Bug 895939 - Click-to-activate plugin notification is ugly, r=jaws a=bbajaj
authorBenjamin Smedberg <benjamin@smedbergs.us>
Fri, 26 Jul 2013 20:52:21 -0400
changeset 148133 fde37f82b690a5b41744cb07bfad3d637ea60ec9
parent 148132 c026ea09decf82145ddb32ef927baec2a6f1b025
child 148134 7fbedcc708697eef8a281f022b0f628a1cae7076
push id2697
push userbbajaj@mozilla.com
push dateMon, 05 Aug 2013 18:49:53 +0000
treeherdermozilla-beta@dfec938c7b63 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjaws, bbajaj
bugs895939
milestone24.0a2
Bug 895939 - Click-to-activate plugin notification is ugly, r=jaws a=bbajaj
browser/themes/osx/browser.css
browser/themes/shared/plugin-doorhanger.inc.css
--- a/browser/themes/osx/browser.css
+++ b/browser/themes/osx/browser.css
@@ -3635,16 +3635,26 @@ toolbarbutton.chevron > .toolbarbutton-m
   font-size: 120%;
 }
 
 %include ../shared/devtools/responsivedesign.inc.css
 %include ../shared/devtools/highlighter.inc.css
 %include ../shared/devtools/commandline.inc.css
 %include ../shared/plugin-doorhanger.inc.css
 
+/* On mac, the popup notification contents are indented by default and so
+  the default closebutton margins from notification.css require adjustment */
+
+.click-to-play-plugins-notification-description-box > .popup-notification-closebutton {
+  -moz-margin-end: -6px;
+  margin-top: -7px;
+}
+
+
+
 .gclitoolbar-input-node > .textbox-input-box > html|*.textbox-input::-moz-selection {
   color: hsl(210,11%,16%);
 }
 
 /* Developer Toolbar */
 
 #developer-toolbar-closebutton {
   margin-left: 8px;
--- a/browser/themes/shared/plugin-doorhanger.inc.css
+++ b/browser/themes/shared/plugin-doorhanger.inc.css
@@ -2,55 +2,52 @@
   padding: 0px;
 }
 
 .click-to-play-plugins-notification-content {
   width: 28em;
 }
 
 .click-to-play-plugins-notification-center-box {
-  background-color: rgba(255,255,255,0.3);
+  border: 1px solid ThreeDShadow;
+  margin: 10px;
 }
 
-.plugin-popupnotification-centeritem {
-  border-top: 1px solid ThreeDShadow;
+.plugin-popupnotification-centeritem:nth-child(odd) {
+  background-color: rgba(0,0,0,0.1);
 }
 
 .center-item-label {
+  -moz-margin-start: 6px;
   margin-bottom: 0;
   text-overflow: ellipsis;
 }
 
 .center-item-warning-icon {
   background-image: url("chrome://mozapps/skin/extensions/alerticon-info-negative.png");
   background-repeat: no-repeat;
   width: 16px;
   height: 15px;
   -moz-margin-start: 6px;
 }
 
 .click-to-play-plugins-notification-button-container {
-  border-top: 1px solid ThreeDShadow;
+  background: linear-gradient(rgba(0,0,0,0.04) 60%, transparent);
+  padding: 10px;
+  margin-top: 5px;
 }
 
 .click-to-play-popup-button {
   width: 50%;
 }
 
 .click-to-play-plugins-notification-description-box {
-  margin-left: 10px;
-  margin-right: 10px;
-  margin-top: 2px;
-  padding-bottom: 2px;
+  padding: 10px;
 }
 
 .click-to-play-plugins-outer-description {
   margin-top: 8px;
 }
 
-.click-to-play-plugins-notification-description-box > .popup-notification-closebutton {
-  margin: 0;
-}
-
 .click-to-play-plugins-notification-link,
 .center-item-link {
   margin: 0;
 }