Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: chrome/content/ui/overlay.xul

Issue 6526453483044864: Issue 2193 - Added notification opt-out (Firefox) (Closed)
Patch Set: Rebased to 5434ef6e1545 Created June 24, 2015, 4:19 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/content/ui/settings.xul » ('j') | lib/ui.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/content/ui/overlay.xul
===================================================================
--- a/chrome/content/ui/overlay.xul
+++ b/chrome/content/ui/overlay.xul
@@ -61,6 +61,7 @@
<menuitem id="abp-status-frameobjects" label="&objecttabs.label;" type="checkbox" command="abp-command-toggleobjtabs"/>
<menuitem id="abp-status-savestats" label="&counthits.label;" type="checkbox" command="abp-command-togglesavestats"/>
<menuitem id="abp-status-sync" label="&sync.label;" type="checkbox" command="abp-command-togglesync"/>
+ <menuitem id="abp-status-shownotifications" label="&shownotifications.label;" type="checkbox" command="abp-command-toggleshownotifications"/>
<menuseparator id="abp-status-iconSettingsSeparator"/>
<menuitem id="abp-status-showintoolbar" label="&showintoolbar.label;" type="checkbox" command="abp-command-toggleshowintoolbar"/>
<menuitem id="abp-status-showinstatusbar" label="&showinstatusbar.label;" type="checkbox" command="abp-command-toggleshowinstatusbar"/>
@@ -93,6 +94,7 @@
<command id="abp-command-enable" oncommand="//"/>
<command id="abp-command-contribute"/>
<command id="abp-command-contribute-hide"/>
+ <command id="abp-command-toggleshownotifications"/>
</commandset>
<statusbarpanel id="abp-status" class="statusbarpanel-iconic"
@@ -121,12 +123,20 @@
<!-- Notification panel -->
<panel id="abp-notification" type="arrow" orient="vertical"
xmlns:html="http://www.w3.org/1999/xhtml">
- <label id="abp-notification-title" class="header"/>
- <html:p id="abp-notification-message"/>
- <hbox id="abp-notification-question">
- <button id="abp-notification-yes" label="&notification.button.yes;"/>
- <button id="abp-notification-no" label="&notification.button.no;"/>
- <toolbarbutton id="abp-notification-close" label="&notification.button.close;"></toolbarbutton>
- </hbox>
+ <vbox id="abp-notification-content">
+ <toolbar align="start">
+ <label id="abp-notification-title" class="header" flex="1" crop="end"/>
+ <toolbarbutton id="abp-notification-close" tooltiptext="&notification.button.close;"/>
+ </toolbar>
+ <html:p id="abp-notification-message"/>
+ <hbox id="abp-notification-question">
+ <button id="abp-notification-yes" label="&notification.button.yes;"/>
+ <button id="abp-notification-no" label="&notification.button.no;"/>
+ </hbox>
+ </vbox>
+ <vbox id="abp-notification-close-content">
+ <toolbarbutton id="abp-notification-hide" label="&notification.closing.button.hide;"/>
+ <toolbarbutton id="abp-notification-optout" label="&notification.closing.button.optout;"/>
+ </vbox>
</panel>
</overlay>
« no previous file with comments | « no previous file | chrome/content/ui/settings.xul » ('j') | lib/ui.js » ('J')

Powered by Google App Engine
This is Rietveld