Closed Bug 547805 Opened 15 years ago Closed 6 years ago

Implement lightweight theming with -moz-element()

Categories

(Firefox :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: mstange, Assigned: mstange)

References

Details

Attachments

(1 file, 2 obsolete files)

When bug 506826 lands we can make LightweightThemeConsumer a little more elegant: Instead of setting the image as background-image on the documentElement and the "footer", we can use mozSetImageElement to make the header and footer images available to be used everywhere.

One advantage of this is that we won't need the lightweightthemesfooter attribute any more and instead have a far more general solution. The other advantage is that this opens up styling possibilities that weren't feasible before:

 1. Fixing the "active tab" problem in the Mac theme:
    Right now we can't give the tab strip a darker background because the
    active tab should be filled with the undarkened original Persona image.
    If we have -moz-element(), we can just set the image on the active tab
    again, thusly covering the darker tabstrip.
    background-attachment: fixed will do the hard positioning work for us.

 2. Blurred button backgrounds:
    At the moment the navbar buttons are pretty opaque in order to make them
    "readable". But this unreadability is primarily caused by noisy background
    textures. If buttons were made of some kind of "glass" that blurred the
    background they could be made less opaque without compromising readability.
    Now that we're converting buttons to use border-radius and stuff, this
    becomes possible pretty easily: LightweightThemeConsumer can just set two
    versions of the persona image using mozSetImageElement - the original one,
    and a blurred one. Then the navbar buttons can do
    background: -moz-element(lwtheme-header-blurred) top right no-repeat fixed;
    and we're done.
Attached patch v1 (obsolete) (deleted) — Splinter Review
Assignee: nobody → mstange
Status: NEW → ASSIGNED
Attachment #472609 - Flags: review?(dao)
Comment on attachment 472609 [details] [diff] [review]
v1

new patch coming
Attachment #472609 - Attachment is obsolete: true
Attachment #472609 - Flags: review?(dao)
Attached patch v2 (obsolete) (deleted) — Splinter Review
Attachment #473477 - Flags: review?(dao)
Attached patch v3 (deleted) — Splinter Review
Dão, I'd really like to get this in, it allows us to make the tabs look so much better... Do you think this affects many addons?
Attachment #473477 - Attachment is obsolete: true
Attachment #497332 - Flags: review?(dao)
Attachment #473477 - Flags: review?(dao)
Comment on attachment 497332 [details] [diff] [review]
v3

Since header images can be quite large, I was concerned about duplicating them negatively affecting tab animations or tab switching performance. However, depending on how efficient -moz-element is, this might still perform better than how we ended up doing it today. But I don't really know that. Needs reliable data.
Attachment #497332 - Flags: review?(dao)
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: