Bug 1231808 - Hide Pause button in Infobar using CSS r=Standard8 a=lizzard
authorDavid Critchley <david@priologic.com>
Tue, 05 Jan 2016 19:17:54 +0000
changeset 310793 06935c4a6dc7dd19003246b60599f0057b1a636e
parent 310792 47c244a99493cabbae2db969257f20fa5e315c8e
child 310794 65635ce66b9eeb2f5c1d08e97d2318bb6d91fe3f
push id5513
push userraliiev@mozilla.com
push dateMon, 25 Jan 2016 13:55:34 +0000
treeherdermozilla-beta@5ee97dd05b5c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersStandard8, lizzard
bugs1231808
milestone45.0a2
Bug 1231808 - Hide Pause button in Infobar using CSS r=Standard8 a=lizzard
browser/extensions/loop/bootstrap.js
browser/extensions/loop/skin/osx/platform.css
browser/extensions/loop/skin/shared/loop.css
--- a/browser/extensions/loop/bootstrap.js
+++ b/browser/extensions/loop/bootstrap.js
@@ -533,26 +533,28 @@ var WindowListener = {
               bar.label = paused ? this._getString("infobar_screenshare_paused_browser_message") :
                 this._getString("infobar_screenshare_browser_message2");
               bar.classList.toggle("paused", paused);
               buttonNode.label = paused ? this._getString("infobar_button_resume_label") :
                 this._getString("infobar_button_pause_label");
               buttonNode.accessKey = paused ? this._getString("infobar_button_resume_accesskey") :
                 this._getString("infobar_button_pause_accesskey");
               return true;
-            }
+            },
+            type: "pause"
           },
           {
             label: this._getString("infobar_button_stop_label"),
             accessKey: this._getString("infobar_button_stop_accesskey"),
             isDefault: true,
             callback: () => {
               this._hideBrowserSharingInfoBar();
               LoopUI.MozLoopService.hangupAllChatWindows();
-            }
+            },
+            type: "stop"
           }]
         );
 
         // Keep showing the notification bar until the user explicitly closes it.
         bar.persistence = -1;
       },
 
       /**
--- a/browser/extensions/loop/skin/osx/platform.css
+++ b/browser/extensions/loop/skin/osx/platform.css
@@ -16,16 +16,21 @@
     background: #ebebeb;
   }
 
   notification[value="loop-sharing-notification"] .notification-button {
     background: #fff;
     border-radius: 0;
   }
 
+  /* Hide Pause/Resume button until the functionality is complete */
+  notification[value="loop-sharing-notification"] .notification-button[type="pause"] {
+    display:none;
+  }
+
   notification[value="loop-sharing-notification"].paused .notification-button {
     background: #57bd35;
   }
 
   notification[value="loop-sharing-notification"].paused .notification-button:hover {
     background: #39a017;
   }
 
--- a/browser/extensions/loop/skin/shared/loop.css
+++ b/browser/extensions/loop/skin/shared/loop.css
@@ -187,16 +187,21 @@
     width: 100px;
     height: 40px;
     margin: 0;
     list-style-image: url(chrome://loop/content/shared/img/pause-12x12.svg);
     box-shadow: 0 -1px 1px rgba(0,0,0,.5) inset;
     text-shadow: none;
   }
 
+  /* Hide Pause/Resume button until the functionality is complete */
+  notification[value="loop-sharing-notification"] .notification-button[type="pause"] {
+    display:none;
+  }
+
   notification[value="loop-sharing-notification"].paused .notification-button {
     background-color: #57bd35;
     color: #fff;
     list-style-image: url(chrome://loop/content/shared/img/play-12x12.svg);
   }
 
   notification[value="loop-sharing-notification"].paused .notification-button:hover {
     background-color: #39a017;