add iconPosition property to touch bar buttons

diff --git a/atom/browser/ui/cocoa/atom_touch_bar.mm b/atom/browser/ui/cocoa/atom_touch_bar.mm
index a3e846df4871fa0a121bc8abdca69b9f941c5878..95f36cf02d027070e65f14699546db5f6996c05d 100644 (file)
--- a/atom/browser/ui/cocoa/atom_touch_bar.mm
+++ b/atom/browser/ui/cocoa/atom_touch_bar.mm
@@ -307,9 +307,18 @@ static NSString* const ImageScrubberItemIdentifier = @"scrubber.image.item";
   settings.Get("label", &label);
   button.title = base::SysUTF8ToNSString(label);
 
+  std::string iconPosition = "left";
+  settings.Get("iconPosition", &iconPosition);
   gfx::Image image;
   if (settings.Get("icon", &image)) {
     button.image = image.AsNSImage();
+    if (iconPosition == "overlay") {
+        button.imagePosition = NSImageOverlaps;
+    } else if (iconPosition == "right") {
+        button.imagePosition = NSImageRight;
+    } else {
+        button.imagePosition = NSImageLeft;
+    }
   }
 }
 

diff --git a/docs/api/touch-bar-button.md b/docs/api/touch-bar-button.md
index 177e51bcdb20019a982f5750100590dcb1a923da..456fc207fe98260b4f971154f9260fad685fc563 100644 (file)
--- a/docs/api/touch-bar-button.md
+++ b/docs/api/touch-bar-button.md
@@ -11,6 +11,7 @@ Process: [Main](../tutorial/quick-start.md#main-process)
   * `backgroundColor` String (optional) - Button background color in hex format,
     i.e `#ABCDEF`.
   * `icon` [NativeImage](native-image.md) (optional) - Button icon.
+  * `iconPosition` String - Can be `left`, `right` or `overlay`.
   * `click` Function (optional) - Function to call when the button is clicked.
 
 ### Instance Properties

diff --git a/lib/browser/api/touch-bar.js b/lib/browser/api/touch-bar.js
index 1a9c3238a97feb95f25fab23e88163252e072518..4bb129449eb9457544ff671f9325b32f3dc450c9 100644 (file)
--- a/lib/browser/api/touch-bar.js
+++ b/lib/browser/api/touch-bar.js
@@ -159,10 +159,11 @@ TouchBar.TouchBarButton = class TouchBarButton extends TouchBarItem {
     super()
     if (config == null) config = {}
     this.type = 'button'
-    const {click, icon, label, backgroundColor} = config
+    const {click, icon, iconPosition, label, backgroundColor} = config
     this._addLiveProperty('label', label)
     this._addLiveProperty('backgroundColor', backgroundColor)
     this._addLiveProperty('icon', icon)
+    this._addLiveProperty('iconPosition', iconPosition)
     if (typeof click === 'function') {
       this.onInteraction = () => {
         config.click()