Closed Bug 479674 Opened 15 years ago Closed 11 years ago

Don't include toolbar.css in application chrome

Categories

(Calendar :: Dialogs, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: dao, Assigned: mschroeder)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #465731 +++

After bug 404229, we don't need toolbar.css just to make our buttons work. See attachment 349324 [details] [diff] [review].
Depends on: 404229
No longer depends on: 465731
Component: General → Sunbird Only
QA Contact: general → sunbird
There is also one occurrence not in Sunbird code: https://mxr.mozilla.org/comm-central/search?string=chrome%3A%2F%2Fglobal%2Fskin%2Ftoolbar.css&case=1&find=%2Fcalendar%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=comm-central
Assignee: nobody → mschroeder
Status: NEW → ASSIGNED
Component: Sunbird Only → Dialogs
Attached patch Patch v1 — — Splinter Review
The toolbox XUL element already loads the toolbar.css.
Attachment #740106 - Flags: review?(philipp)
Attachment #740106 - Flags: review?(philipp) → review+
https://hg.mozilla.org/comm-central/rev/c6407f244d72
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.5
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: