Closed
Bug 510442
Opened 15 years ago
Closed 15 years ago
remove unneeded CSS and image data for toolbar buttons that are always enabled
Categories
(Firefox :: Theme, defect)
Firefox
Theme
Tracking
()
RESOLVED
FIXED
Firefox 3.7a1
| Tracking | Status | |
|---|---|---|
| status1.9.2 | --- | beta1-fixed |
People
(Reporter: dao, Assigned: dao)
Details
(Keywords: perf)
Attachments
(1 file)
|
248.57 KB,
patch
|
rflint
:
review+
beltzner
:
approval1.9.2+
|
Details | Diff | Splinter Review |
No description provided.
|
Assignee | |
Comment 1•15 years ago
|
||
|
|
Assignee | |
Updated•15 years ago
|
Attachment #394812 -
Flags: review? → review?(rflint)
|
|
Assignee | |
Comment 2•15 years ago
|
||
I haven't removed the disabled icons from pinstripe's Toolbar.png, as bug 504160 may use them.
|
||
Updated•15 years ago
|
Attachment #394812 -
Flags: review?(rflint) → review+
|
|
Assignee | |
Comment 3•15 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/3c4645c657ad
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a1
|
|
Assignee | |
Updated•15 years ago
|
Summary: For toolbar buttons that won't ever be disabled, remove disabled icons from Toolbar.png and the corresponding CSS code too → remove unneeded CSS and image data for toolbar buttons that are always enabled
|
|
Assignee | |
Updated•15 years ago
|
Attachment #394812 -
Flags: approval1.9.2?
|
||
Comment 4•15 years ago
|
||
CC'ing faaborg, who might want to update his icon inventory.
|
||
Comment 5•15 years ago
|
||
Comment on attachment 394812 [details] [diff] [review] patch a192=beltzner
Attachment #394812 -
Flags: approval1.9.2? → approval1.9.2+
|
|
Assignee | |
Updated•15 years ago
|
Keywords: checkin-needed
|
|
Assignee | |
Comment 6•15 years ago
|
||
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/e62f3d14a588
status1.9.2:
--- → beta1-fixed
Keywords: checkin-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•