Closed Bug 1727632 Opened 3 years ago Closed 3 years ago

Create a prototype theme generator module

Categories

(Firefox :: Theme, enhancement)

enhancement
Points:
3

Tracking

()

RESOLVED WONTFIX
Iteration:
93.2 - Aug 23 - Sep 5

People

(Reporter: bugzilla, Assigned: bugzilla)

References

Details

(Whiteboard: [fidefe-theme])

Attachments

(1 file, 1 obsolete file)

We may need to create a theme generator module to make monochromatic themes more consistent and maintainable. We should start with a rudimentary prototype to investigate approach and find possible shortcomings.

Dao and Amy, I'd be grateful for some early feedback on this approach to a theme generator. This patch defines that we want to use nine colours in monochromatic themes and maps each theme property to one of those nine colours. Then, a module outputs a color-mix value using the level of contrast requested and the theme's base color. It's pretty rudimentary -- the module outputs a simple string, and doesn't yet account for the other kinds of themes UX wants from this project. Still, it automates creating theme manifests so we don't have to manually create near-copies of browser/themes/addons/monochromatic-purple/manifest.json using different base colors. The set of 9 contrast ratios and how they map to theme properties is pretty much arbitrary. We'll need UX input on the logic to use there.

If you want to test the patch, toggle browser.theme.temporary.monochromatic.enabled and restart Firefox to get the Monochromatic Purple theme to appear. You could see how it works with colors other than purple by changing the value of frame in the theme manifest. I've attached a screenshot of how it looks for #D0FFED.

I'm OoO for the rest of the week, so no rush on offering feedback!

Flags: needinfo?(dao+bmo)
Flags: needinfo?(achurchwell)
Whiteboard: [fidefe-theme]
Flags: needinfo?(achurchwell)

This is no longer something we're pursuing. I don't think UX would want to be constrained by a generator when designing future themes.

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Flags: needinfo?(dao+bmo)
Resolution: --- → WONTFIX
Attachment #9238066 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: