Closed
Bug 565300
Opened 15 years ago
Closed 14 years ago
Update category names to match those from the design document
Categories
(Toolkit :: Add-ons Manager, defect)
Toolkit
Add-ons Manager
Tracking
()
VERIFIED
FIXED
mozilla2.0b7
Tracking | Status | |
---|---|---|
blocking2.0 | --- | beta7+ |
People
(Reporter: whimboo, Assigned: mossop)
References
()
Details
(Whiteboard: [rewrite][strings])
Attachments
(1 file)
2.50 KB,
patch
|
Unfocused
:
review+
|
Details | Diff | Splinter Review |
Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.3a5pre) Gecko/20100511 Minefield/3.7a5pre ID:20100511030640
On bug 557240 we switched back to the category naming for older releases. Given by the design documents we will have to use the new names at some point. This bug should track that work.
Category names right now are:
* Get Add-ons
* Languages
* Search Engines
* Extensions
* Appearance
* Plugins
* Updates
Adding uiwanted to make sure we will get the final list of category names.
Comment 1•15 years ago
|
||
There's been a lot of discussion specifically about the naming of the "Get Add-ons"/"Discover" pane. I'll summarize:
* "Get Add-ons" is seen as more descriptive, and potentially a lot easier to understand for people who won't know much about addons
* "Discover" conflicts with addons that provide discovery services
* There have been troubles localizing "Get Add-ons" - whereas "Discover" is seen as an easier phrase to localize while keeping its meaning
Comment 2•15 years ago
|
||
Marking dup of Bug 553473. As Blair said, we're keeping "Get-Addons" now
Reporter | ||
Comment 3•15 years ago
|
||
This bug is not only about "Get Addons" vs. Discover. Bug 553473 is definitely a part of it and I was aware of that bug when I have filed this one. Comparing the current design docs with the implemented category names shows a difference for the Themes vs. Appearances panel name. The other ones look fine but would need a clear definition we can work with.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Assignee | ||
Updated•15 years ago
|
blocking2.0: --- → final+
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → bmcbride
Comment 6•14 years ago
|
||
The only string change needed here is "Themes" -> "Appearance" (but it needs changed in 2 places).
Sadly, changing the entity name in extensions.properties is painful/not sanely possible while the appearance pane is implemented by the list view (since it determines the entity name programatically). Thankfully, this is changing (by work done in bug 520124), so once that's done, this will be trivially easy.
Comment 7•14 years ago
|
||
Er, and by "painful", I mean without special casing it. Its not possible to do it *nicely*.
Assignee | ||
Comment 8•14 years ago
|
||
Well I was actually making that change in bug 520124 so I might as well just own this for now.
Assignee: bmcbride → dtownsend
Depends on: 520124
Assignee | ||
Updated•14 years ago
|
blocking2.0: beta5+ → beta6+
Assignee | ||
Comment 9•14 years ago
|
||
Now that we aren't making the new appearance pane do we still want to name it Appearance or do we stick with Themes?
Keywords: uiwanted
Comment 10•14 years ago
|
||
(In reply to comment #9)
> Now that we aren't making the new appearance pane do we still want to name it
> Appearance or do we stick with Themes?
Its still about changing the appearance, and not solely about normal themes. The purpose hasn't changed, just the UI for it. So I'd say keep it as Appearance.
Assignee | ||
Comment 11•14 years ago
|
||
Well now the header text is gone this is somewhat easier to do anyway.
Attachment #470986 -
Flags: review?(bmcbride)
Assignee | ||
Updated•14 years ago
|
Status: REOPENED → ASSIGNED
Updated•14 years ago
|
Attachment #470986 -
Flags: review?(bmcbride) → review+
Assignee | ||
Updated•14 years ago
|
Whiteboard: [rewrite] → [rewrite][strings]
Assignee | ||
Comment 12•14 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 15 years ago → 14 years ago
Flags: in-testsuite-
Flags: in-litmus-
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b6
Reporter | ||
Comment 13•14 years ago
|
||
Verified fixed with builds on all platforms like Mozilla/5.0 (Macintosh; Intel Mac OS X 10.5; rv:2.0b6pre) Gecko/20100902 Firefox/4.0b6pre
Status: RESOLVED → VERIFIED
Keywords: uiwanted
You need to log in
before you can comment on or make changes to this bug.
Description
•