Closed
Bug 547805
Opened 15 years ago
Closed 6 years ago
Implement lightweight theming with -moz-element()
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: mstange, Assigned: mstange)
References
Details
Attachments
(1 file, 2 obsolete files)
7.70 KB,
patch
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•14 years ago
|
||
Assignee | ||
Comment 2•14 years ago
|
||
Comment on attachment 472609 [details] [diff] [review]
v1
new patch coming
Attachment #472609 -
Attachment is obsolete: true
Attachment #472609 -
Flags: review?(dao)
Assignee | ||
Comment 3•14 years ago
|
||
Attachment #473477 -
Flags: review?(dao)
Assignee | ||
Comment 4•14 years ago
|
||
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 5•11 years ago
|
||
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)
Assignee | ||
Updated•6 years ago
|
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.
Description
•