Closed Bug 508739 Opened 16 years ago Closed 16 years ago

Replace many gradient images with CSS gradients

Categories

(Toolkit :: Themes, defect)

All
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.2b1

People

(Reporter: mstange, Assigned: mstange)

References

Details

(Keywords: memory-footprint)

Attachments

(1 file, 1 obsolete file)

Attached patch v1 (obsolete) — Splinter Review
"Many" because I probably haven't found all of them. This patch also reworks some buttons while I'm there.
Attachment #392881 - Flags: review?(dao)
Could the colors be #defined in some central place or could the background rules be grouped in global classes? Hard-coded colors tend to get out of sync if they are scattered all over the place.
Blocks: 508940
I can do that but I'd like to break it out into a separate bug. It's not that straightforward. I've filed bug 508940.
Attached patch v2Splinter Review
removes double margin
Attachment #393123 - Flags: review?(dao)
Attachment #392881 - Attachment is obsolete: true
Attachment #392881 - Flags: review?(dao)
Attachment #393123 - Flags: review?(dao) → review+
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2b1
Depends on: 510426
Depends on: 529542
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: