Closed
Bug 1344058
Opened 8 years ago
Closed 3 years ago
Add-on/extension (e.g. default theme) name and description not localized by pref if its disabled, affects customization e.g. in Firefox Developer Edition
Categories
(Toolkit :: Add-ons Manager, defect, P5)
Toolkit
Add-ons Manager
Tracking
()
RESOLVED
INACTIVE
People
(Reporter: aryx, Unassigned)
References
Details
(Whiteboard: triaged)
E.g. latest 53.0a2 on Windows 8.1
Add-ons which have a localized name and description and this setting
>pref("extensions.<id>.name/description", "chrome://<path>/<file>.properties");
and
>extensions.{972ce4c6-7e08-4474-a285-3208198ce6fd}.name/description=My Extension Name
in the file don't have it localized if it's disabled (at least after a restart).
This also affects the default theme.
Steps to reproduce:
1. Use a localized build (developer edition is the easiest).
2. If you don't have the developer edition, open the Add-ons Manager and install a theme from the top-most pane.
Actual result:
- In the theme list, the default theme will now have an English name and description (works the first time, sometimes later it might a require a restart).
- In the customization mode from the hamburger menu on the right side of the toolbar, these English strings are used as label and tooltip in the theme menu at the bottom.
Comment 1•8 years ago
|
||
in 57 this will work with webextensions.
Status: NEW → UNCONFIRMED
Ever confirmed: false
Flags: needinfo?(amckay)
Priority: -- → P5
Comment 2•8 years ago
|
||
Sorry, to be clearer, will this be relevant in Desktop release once we only support WebExtensions? I don't think so.
Flags: needinfo?(amckay)
Comment 3•8 years ago
|
||
(In reply to Andy McKay [:andym] from comment #2)
> Sorry, to be clearer, will this be relevant in Desktop release once we only
> support WebExtensions? I don't think so.
I think it will, because the default theme will still be affected per comment #0, and at least for the moment I expect that'll keep being reflected as a 'complete' theme in the add-ons manager. I would be surprised if we were able to strip all the relevant code out immediately for 57, but maybe I'm wrong. Dave would have a better idea about this.
Flags: needinfo?(dtownsend)
Comment 4•8 years ago
|
||
Once complete themes are gone this shouldn't be an issue but we should verify that.
Flags: needinfo?(dtownsend)
Comment 5•8 years ago
|
||
checking what our default theme is going to be with Photon- to understand what different kindso are.
Flags: needinfo?(rhelmer)
Whiteboard: investigating
Comment 6•8 years ago
|
||
rob will ask mossop what he meant - think won't be issue beyond 57
Comment 7•8 years ago
|
||
rob have you talked to mossop
Updated•8 years ago
|
Flags: needinfo?(rhelmer)
Comment 8•7 years ago
|
||
Sorry for the delay. I believe this will still be an issue post-57 (what Gijs says in comment 3) because Dave is right in comment 4 that removing complete theme support will fix this but I don't know that we're going to have that removed in time for 57 (seems like the minimum bar would be to refuse to load themes for 57, and not accept them on AMO anymore etc)
I'm assuming this is a long-standing bug and while annoying is pretty low in severity, so while we probably won't get the above done in time for 57 I think it will go away soon since we definitely want to remove all the complete theme code from the addons manager.
Updated•7 years ago
|
Whiteboard: investigating → triaged
Comment 10•4 years ago
|
||
The only way I've been able to reproduce this is by installing a new language in about:preferences, which seems to not require a restart. Then I see english names for themes and buttons in customization mode. Any other steps I attempted I could not reproduce a mix. It would be great if someone could reproduce this. I'd close this, but am not certain it is not reproducible.
Severity: normal → S4
Comment 11•3 years ago
|
||
name/description cannot be changed via prefs any more since the removal of the feature in bug 1462840.
Status: UNCONFIRMED → RESOLVED
Closed: 3 years ago
Resolution: --- → INACTIVE
You need to log in
before you can comment on or make changes to this bug.
Description
•