1355788 - rename "Appearance" to "Themes"

Status: RESOLVED FIXED

(Toolkit :: Add-ons Manager, enhancement, P4)

Description

[Markus Jaritz [:designakt] (UX)]

The add-ons managers left hand side menu is the only place that uses the word "Appearance" when referring to "Themes", everywhere else we use the word "Themes". Therefor we should also use "Themes" in the add-ons manager menu to be consistent.

Jorge has confirmed this in 
https://bugzilla.mozilla.org/show_bug.cgi?id=1254301#c12

Comment 1

[Andy McKay]

Attachment: bug 1355788 rename Appearance to Themes

Review commit: https://reviewboard.mozilla.org/r/172208/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/172208/

Comment 2

[Andy McKay]

Comment on attachment 8900752 [details]
bug 1355788 rename Appearance to Themes

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/172208/diff/1-2/

Comment 3

[Francesco Lodolo [:flod]]

Comment on attachment 8900752 [details]
bug 1355788 rename Appearance to Themes

https://reviewboard.mozilla.org/r/172208/#review177458

::: toolkit/locales/en-US/chrome/mozapps/extensions/extensions.properties:185
(Diff revision 2)
>  
>  #LOCALIZATION NOTE (eulaHeader) %S is name of the add-on asking the user to agree to the EULA
>  eulaHeader=%S requires that you accept the following End User License Agreement before installation can proceed:
>  
>  type.extension.name=Extensions
> -type.theme.name=Appearance
> +type.theme.name=Themes

Sorry, you need a new ID for this type of changes
https://developer.mozilla.org/en-US/docs/Mozilla/Localization/Localization_content_best_practices#Changing_existing_strings

Comment 4

[Andrew Swan [:aswan]]

(In reply to Francesco Lodolo [:flod] from comment #3)
> Sorry, you need a new ID for this type of changes

And the ids that we use here are programmatically generated :(

Comment 5

[Francesco Lodolo [:flod]]

(In reply to Andrew Swan [:aswan] from comment #4)
> (In reply to Francesco Lodolo [:flod] from comment #3)
> > Sorry, you need a new ID for this type of changes
> 
> And the ids that we use here are programmatically generated :(

Can you point me to the code? It would be strange if there's no way to map it to a different ID.

By changing the string like this, there's basically no way to ensure that the content will be ever updated. Localizers won't even be aware of the change, because the string is already translated in their file.

Comment 6

[Andy McKay]

Comment on attachment 8900752 [details]
bug 1355788 rename Appearance to Themes

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/172208/diff/2-3/

Comment 7

[Andy McKay]

There was at least one other place that depended upon the value of aID being theme. It seemed to just make all this simpler if I just remove the dynamically generated IDs. Something that :flod said was bad anyways :)

Comment 8

[Andrew Swan [:aswan]]

Comment on attachment 8900752 [details]
bug 1355788 rename Appearance to Themes

https://reviewboard.mozilla.org/r/172208/#review177452

I assume you tested this by hand?
Also, there's (at least?) one more provider that you didn't change in `browser/modules/Experiments.jsm`

Comment 9

[Andy McKay]

Comment on attachment 8900752 [details]
bug 1355788 rename Appearance to Themes

https://reviewboard.mozilla.org/r/172208/#review177452

I did, I ran a few tests even. Thanks for spotting Experiments.jsm, I've added that in.

Comment 10

[Andy McKay]

Comment on attachment 8900752 [details]
bug 1355788 rename Appearance to Themes

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/172208/diff/3-4/

Comment 11

[Andy McKay]

Comment on attachment 8900752 [details]
bug 1355788 rename Appearance to Themes

https://reviewboard.mozilla.org/r/172208/#review177458

> Sorry, you need a new ID for this type of changes
> https://developer.mozilla.org/en-US/docs/Mozilla/Localization/Localization_content_best_practices#Changing_existing_strings

Changed from type.theme to type.themes

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/03f4526ce1e7
rename Appearance to Themes r=aswan

Comment 13

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/03f4526ce1e7