Closed Bug 1160731 Opened 10 years ago Closed 10 years ago

Get rid of inContentUI.css

Categories

(Toolkit :: Themes, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla41
Tracking Status
firefox41 --- fixed

People

(Reporter: Paenglab, Assigned: Paenglab)

References

Details

Attachments

(1 file, 1 obsolete file)

Actually we have two in-content style-sheets, the old in global/skin/inContentUI.css and the new in global/skin/in-content/common.css. A lot of content is now using the new one. With the add-on manager the biggest user has moved now to the new style-sheet. The remaining pages are: about:sync-progress, about:permissions, about:downloads and requestAutocomplete. When this pages have moved to global/skin/in-content/common.css, we can remove the inContentUI.css.
Attached patch Bug1160731.patch (obsolete) — Splinter Review
Now there is no more consumer of inContentUI.css and it can be removed.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8607082 - Flags: review?(dolske)
Attachment #8607082 - Flags: review?(dolske) → review?(jaws)
Attachment #8607082 - Flags: review?(jaws) → review+
Attached patch Bug1160731.patchSplinter Review
Updated the commit message to r=jaws
Attachment #8607082 - Attachment is obsolete: true
Attachment #8608012 - Flags: review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: