Closed
Bug 753618
Opened 12 years ago
Closed 12 years ago
Remove outdated default theme icons and previews from Add-ons Manager
Categories
(Firefox :: Theme, defect)
Firefox
Theme
Tracking
()
RESOLVED
FIXED
Firefox 15
People
(Reporter: fryn, Assigned: fryn)
References
Details
(Whiteboard: [fixed-in-fx-team])
Attachments
(1 file, 2 obsolete files)
9.53 KB,
patch
|
Unfocused
:
review+
|
Details | Diff | Splinter Review |
Jared reminded me that the icons and preview images for the default themes in the Add-ons Manager are long outdated. I removed the preview images and replaced the icons with 11-byte transparent 1x1 PNGs to avoid having to change any JS for now. We can revamp the Add-ons Manager UI at a later time.
Attachment #622601 -
Flags: review?(jaws)
Assignee | ||
Comment 1•12 years ago
|
||
Now using the generic theme icon as the default theme icon for all 3 platforms.
Attachment #622601 -
Attachment is obsolete: true
Attachment #622601 -
Flags: review?(jaws)
Attachment #622602 -
Flags: review?(jaws)
Is it true Default theme would be removed from the list ? If so then how would a user return to default theme when using some other theme. Though australis mockups show a new customisation page.
Assignee | ||
Comment 3•12 years ago
|
||
Forgot to edit browser/installer/package-manifest.in. Pushing to try and locally testing to make sure everything works.
Attachment #622602 -
Attachment is obsolete: true
Attachment #622602 -
Flags: review?(jaws)
Attachment #622605 -
Flags: review?(jaws)
Comment 4•12 years ago
|
||
(In reply to bogas04 from comment #2) > Is it true Default theme would be removed from the list ? If so then how > would a user return to default theme when using some other theme. As part of the revamp of the Add-on Manager's appearance pane (bug 520124), I've been planning to only show the default theme when there are other heavyweight themes installed. So by default, the default theme will be hidden.
Comment 5•12 years ago
|
||
Though I'm a bit confused by this bug. Why did you initially want to remove the icons now? The preview is indeed very outdated, but why not replace it with an updated one? Note that once bug 520124 eventually lands the icon won't be used, but the preview image will. See attachment 464426 [details] for a rough idea.
Comment 6•12 years ago
|
||
Comment on attachment 622605 [details] [diff] [review] patch v3 (Pretty sure I should review this, given it exclusively affects the Addons Manager.)
Attachment #622605 -
Flags: review?(jaws) → review?(bmcbride)
Assignee | ||
Comment 7•12 years ago
|
||
(In reply to Blair McBride (:Unfocused) from comment #5) > Though I'm a bit confused by this bug. Why did you initially want to remove > the icons now? The preview is indeed very outdated, but why not replace it > with an updated one? Because the theme is continually changing and will continue to do so for the foreseeable future, so the image will almost always be out-of-date. It's also not especially useful to display the preview of the default theme which the user already knows it looks like. > Note that once bug 520124 eventually lands the icon won't be used, but the > preview image will. See attachment 464426 [details] for a rough idea. It looks good. Like we currently do, we should be able to fall back to displaying something generic if a preview is not required. For the longer-term future, the architecture of non-lightweight themes requiring a replacement for every single image that we have in the default theme inherently makes it difficult for potential theme authors to learn how to make a theme and be able to make one quickly. Also, compatibility has been and will be very difficult to maintain. Rather, I think we should make it easy to build restartless extensions that solely apply stylesheets to the browser and replace images. If/as we move to the model that I just described, it will make less and less sense for the default theme to be displayed in the themes tab, since non-lightweight themes will be superseded by restartless extensions using theming modules, and they will be applied on top of the default theme, not replacing the default theme.
Comment 8•12 years ago
|
||
(In reply to Frank Yan (:fryn) from comment #7) > Because the theme is continually changing and will continue to do so for the > foreseeable future, so the image will almost always be out-of-date. Fair enough. > If/as we move to the model that I just described, it will make less and less > sense for the default theme to be displayed in the themes tab, since > non-lightweight themes will be superseded by restartless extensions using > theming modules, and they will be applied on top of the default theme, not > replacing the default theme. Yep, I've been thinking the same thing. The Add-ons Manager won't be dropping support for heavyweight themes though, just deemphasising them - so there will still be cases when the default theme is shown.
Updated•12 years ago
|
Attachment #622605 -
Flags: review?(bmcbride) → review+
Assignee | ||
Comment 9•12 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/53665792bf24
Whiteboard: [fixed-in-fx-team]
Comment 10•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/53665792bf24
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 15
You need to log in
before you can comment on or make changes to this bug.
Description
•