623825 - Remove "Options" button for persona themes

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Tony Chung [:tchung]]

Attachment: theme options button

The fennec addons manager has an options button present for personas themes.   This doesnt seem necessary since persona themes dont have options.  Most themes i've installed have the button grayed out anyway, but there are occasional themes that has the button enabled... which clicking it does nothing anyway.

I suggesting removing options for persona themes altogether.  In desktop firefox, thats not even there anyway.

See screenshot

Repro:
1) install android nightly: Mozilla/5.0 (Android; Linux armv71; rv:2.0b9pre) Gecko/20110106 Firefox/4.0b9pre Fennec/4.0b4pre	
2) install mobile personas extension from AMO
3) wear a few personas.  Wear "Japanese Flowers" for example
4) Open prefs > Addons manager
5) Verify personas themes have an options button that does nothing.  

Expectd:
- there are no options for personas, so should just remove them for these theme types

Actual:
- option button visible, does nothing when clicked.

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

Fabrice, If it makes things easier, let's add an attribute to the XBL, hideoptions="true", that would forcibly hide the Options button. Then the add-on could use it for themes.

Or maybe you have a different idea that doesn't require any Fennec code changes. I just wanted to say that we could add support for it to Fennec - not a problem.

Comment 2

[[:fabrice] Fabrice Desré]

Attachment: patch

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 502047 [details] [diff] [review]
patch

No {} needed on the if stmt

Comment 4

[[:fabrice] Fabrice Desré]

Attachment: updated patch

addressing comment

Comment 5

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed:
http://hg.mozilla.org/mobile-browser/rev/5e1e0d4fb38b

Comment 6

[Tony Chung [:tchung]]

Verified fix on Mozilla/5.0 (Android; Linux armv71; rv:2.0b9pre) Gecko/20110110 Firefox/4.0b9pre Fennec/4.0b4pre