Closed Bug 753618 Opened 9 years ago Closed 9 years ago

Remove outdated default theme icons and previews from Add-ons Manager

Categories

(Firefox :: Theme, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 15

People

(Reporter: fryn, Assigned: fryn)

References

Details

(Whiteboard: [fixed-in-fx-team])

Attachments

(1 file, 2 obsolete files)

Attached patch patch (obsolete) — 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)
Attached patch patch v2 (obsolete) — Splinter Review
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.
Attached patch patch v3Splinter Review
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)
(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.
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 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)
(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.
(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.
Attachment #622605 - Flags: review?(bmcbride) → review+
https://hg.mozilla.org/mozilla-central/rev/53665792bf24
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 15
You need to log in before you can comment on or make changes to this bug.