Bug 1403733 - Fix single button styling on Flash doorhanger r?felipe
There's not a great selector that covers the case of either button
being hidden, so the best option seems to be an additional
attribute.
MozReview-Commit-ID: CMdjWQnPjGA
--- a/browser/base/content/urlbarBindings.xml
+++ b/browser/base/content/urlbarBindings.xml
@@ -2990,16 +2990,22 @@ file, You can obtain one at http://mozil
this._secondaryButton.label = gNavigatorBundle.getString(button2.label);
this._secondaryButton.accessKey = gNavigatorBundle.getString(button2.accesskey);
this._secondaryButton.setAttribute("action", button2.action);
if (button1.default) {
this._primaryButton.setAttribute("default", "true");
} else if (button2.default) {
this._secondaryButton.setAttribute("default", "true");
}
+
+ if (this._primaryButton.hidden) {
+ this._secondaryButton.setAttribute("alone", "true");
+ } else if (this._secondaryButton.hidden) {
+ this._primaryButton.setAttribute("alone", "true");
+ }
]]></body>
</method>
<method name="_setupDescription">
<parameter name="baseString" />
<parameter name="pluginName" /> <!-- null for the multiple-plugin case -->
<parameter name="prePath" />
<body><![CDATA[
var span = document.getAnonymousElementByAttribute(this, "anonid", "click-to-play-plugins-notification-description");
--- a/toolkit/themes/shared/popupnotification.inc.css
+++ b/toolkit/themes/shared/popupnotification.inc.css
@@ -73,17 +73,17 @@
box-shadow: 0 1px 0 hsla(210,4%,10%,.05) inset;
}
.popup-notification-button[disabled] {
background-color: var(--arrowpanel-dimmed-further);
color: graytext;
}
-.popup-notification-button[default] {
+.popup-notification-button[default]:not([alone]) {
flex: 0 50%;
}
.popup-notification-button[default][highlight="true"]:not([disabled]) {
background-color: #0996f8;
color: white;
}
