Bug 623825 - Remove 'Options' button for persona themes [r=mfinkle]
authorFabrice Desré <fabrice@mozilla.com>
Fri, 07 Jan 2011 14:56:56 -0500
changeset 67228 38fc55b950f945b9d0405d74180a4f4b27f51dad
parent 67227 2f30a14583e85f48e76a13defdd59c3cee2ad6ac
child 67229 9c8cc7f65b66b8781ffdf814c8bba6453587010e
push id1
push userroot
push dateTue, 26 Apr 2011 22:38:44 +0000
treeherdermozilla-beta@bfdb6e623a36 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmfinkle
bugs623825
Bug 623825 - Remove 'Options' button for persona themes [r=mfinkle]
mobile/chrome/content/bindings/extensions.xml
--- a/mobile/chrome/content/bindings/extensions.xml
+++ b/mobile/chrome/content/bindings/extensions.xml
@@ -58,16 +58,21 @@
 
           let appDisabled = this.getAttribute("appDisabled");
           if (appDisabled == "true")
             document.getAnonymousElementByAttribute(this, "anonid", "enable-button").setAttribute("disabled", "true");
 
           let isReadOnly = this.getAttribute("isReadonly");
           if (isReadOnly == "true")
             document.getAnonymousElementByAttribute(this, "anonid", "uninstall-button").setAttribute("disabled", "true");
+          
+          // no options for themes
+          // use the unlocalized property, not the typeLabel attribute
+          if (this.addon.type == "theme")
+            document.getAnonymousElementByAttribute(this, "anonid", "options-button").hidden = true;
         ]]>
       </constructor>
 
       <method name="hideOptions">
         <body>
           <![CDATA[
             let box = document.getAnonymousElementByAttribute(this, "anonid", "options-box");
             if (!box.collapsed)