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

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

Issue 5181328018374656: A better approach to make Contribute button Australis-compatible (Closed)
Patch Set: Created Nov. 21, 2013, 12:49 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 | lib/ui.js » ('j') | no next file with comments »
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
@@ -65,18 +65,20 @@
<menuseparator id="abp-status-iconSettingsSeparator"/>
<menuitem id="abp-status-showinaddonbar" label="&showinaddonbar.label;" type="checkbox" command="abp-command-toggleshowintoolbar"/>
<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"/>
</menupopup>
</menu>
<hbox class="abp-contributebutton" id="abp-status-contributebutton" pack="center" align="center">
- <button class="abp-contributebutton-btn" label="&contribute.label;" command="abp-command-contribute" flex="1"/>
- <toolbarbutton class="abp-contributebutton-close" command="abp-command-contribute-hide"/>
+ <!-- noautoclose attribute tells Australis menu that it shouldn't close when
+ these are clicked, see https://bugzilla.mozilla.org/show_bug.cgi?id=940693 -->
+ <button class="abp-contributebutton-btn" label="&contribute.label;" command="abp-command-contribute" flex="1" noautoclose="true"/>
+ <toolbarbutton class="abp-contributebutton-close" command="abp-command-contribute-hide" noautoclose="true"/>
</hbox>
</menupopup>
<keyset id="abp-keyset"/>
<!-- Dummy oncommand attributes are work-arounds for bug 371900 -->
<commandset id="abp-commandset">
<command id="abp-command-sendReport" oncommand="//"/>
« no previous file with comments | « no previous file | lib/ui.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld