Bug 1392979 - Modernize buttons for Flash CTP r=Felipe

This is just the most conservative thing we can do to get the
Flash doorhanger to not look out of place next to other
doorhangers. This does not change the doorhanger's strings or
anything substantive. Still looking into what that would entail,
but I thought I would drop this in as a baseline.

Minor notes: added default="true" to get the OS-specific button
reordering of normal doorhangers, and removed the close button
since the doorhanger will close by just clicking out of it.

MozReview-Commit-ID: 9jhHLfzSeXZ

--HG--
extra : rebase_source : b206531f0b916502bc4917ae93f1e95ec9909c8e
This commit is contained in:
Doug Thayer 2017-08-29 10:56:42 -07:00
Родитель 3d0c3afa5b
Коммит 1206d529cb
2 изменённых файлов: 12 добавлений и 10 удалений

Просмотреть файл

@ -2694,17 +2694,15 @@ file, You can obtain one at http://mozilla.org/MPL/2.0/.
<binding id="click-to-play-plugins-notification" extends="chrome://global/content/bindings/notification.xml#popup-notification">
<content align="start" style="width: &pluginNotification.width;;">
<xul:vbox flex="1" align="stretch" class="popup-notification-main-box"
<xul:vbox flex="1" align="stretch" class="click-to-play-plugins-notification-main-box"
xbl:inherits="popupid">
<xul:hbox class="click-to-play-plugins-notification-description-box" flex="1" align="start">
<xul:description class="click-to-play-plugins-outer-description" flex="1">
<html:span anonid="click-to-play-plugins-notification-description" />
<xul:label class="text-link click-to-play-plugins-notification-link" anonid="click-to-play-plugins-notification-link" />
<html:br/>
<xul:label class="text-link click-to-play-plugins-notification-link popup-notification-learnmore-link"
anonid="click-to-play-plugins-notification-link" />
</xul:description>
<xul:toolbarbutton anonid="closebutton"
class="messageCloseButton popup-notification-closebutton tabbable close-icon"
xbl:inherits="oncommand=closebuttoncommand"
tooltiptext="&closeNotification.tooltip;"/>
</xul:hbox>
<xul:grid anonid="click-to-play-plugins-notification-center-box"
class="click-to-play-plugins-notification-center-box">
@ -2726,11 +2724,12 @@ file, You can obtain one at http://mozilla.org/MPL/2.0/.
class="click-to-play-plugins-notification-button-container"
pack="center" align="center">
<xul:button anonid="primarybutton"
class="click-to-play-popup-button"
class="click-to-play-popup-button popup-notification-button"
oncommand="document.getBindingParent(this)._onButton(this)"
flex="1"/>
<xul:button anonid="secondarybutton"
class="click-to-play-popup-button"
default="true"
class="click-to-play-popup-button popup-notification-button"
oncommand="document.getBindingParent(this)._onButton(this);"
flex="1"/>
</xul:hbox>

Просмотреть файл

@ -27,9 +27,8 @@
.click-to-play-plugins-notification-button-container {
background-color: var(--arrowpanel-dimmed);
border-top: 1px solid var(--panel-separator-color);
padding: 10px;
margin-top: 5px;
display: flex;
}
.click-to-play-popup-button {
@ -49,6 +48,10 @@
margin: 0;
}
.click-to-play-plugins-notification-main-box {
width: 100%;
}
.messageImage[value="plugin-hidden"] {
list-style-image: url(chrome://browser/skin/notification-icons/plugin.svg);
}