Closed
Bug 493812
Opened 13 years ago
Closed 13 years ago
Some css file have selectors related to classes\rules that don't exist anymore
Categories
(Calendar :: Calendar Frontend, defect)
Calendar
Calendar Frontend
Tracking
(Not tracked)
RESOLVED
FIXED
1.0b1
People
(Reporter: bv1578, Assigned: bv1578)
Details
Attachments
(1 file)
3.35 KB,
patch
|
mschroeder
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; it-IT; rv:1.9.0.10) Gecko/2009042316 Firefox/3.0.10 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; it; rv:1.9.1b5pre) Gecko/20090518 Lightning/1.0pre Shredder/3.0b3pre Patch attachment 351864 [details] [diff] [review], that fixed bug 447683, deleted binding "calendar-month-view-column-header" and some xul elements with CSS classes in calendar-month-view.xml file, but in calendar-views.css and calendar-view-bindings.css files there still are selectors that refer to those elements. I have found at least three elements: calendar-month-view-column-header (http://hg.mozilla.org/comm-central/rev/fe713242a103#l4.7) calendar-month-view-column-header-container (http://hg.mozilla.org/comm-central/rev/fe713242a103#l4.9) calendar-month-view-column-header-label (http://hg.mozilla.org/comm-central/rev/fe713242a103#l4.10) Reproducible: Always Steps to Reproduce: 1. search three classes mentioned above on Comm-Central repository or on http://mxr.mozilla.org/comm-central/source/calendar/ Actual Results: you will get only links to css files and no link to xml\xul files. Expected Results: every selector should refer to a binding\class\ID\... defined inside a xul\xml file.
Comment 1•13 years ago
|
||
Confirming... first step to clean up our CSS files. I suspect there is more obsolete CSS lying around.
Severity: minor → trivial
Status: UNCONFIRMED → NEW
Component: General → Calendar Views
Ever confirmed: true
QA Contact: general → views
If mentioned selectors in comment #0 can be deleted with any other considerations, this patch should be right.
Attachment #379242 -
Flags: review?(mschroeder)
Updated•13 years ago
|
Assignee: nobody → bv1578
Status: NEW → ASSIGNED
Comment 3•13 years ago
|
||
Comment on attachment 379242 [details] [diff] [review] fix for the issue in comment #0 Removing these selectors should be fine! r=mschroeder
Attachment #379242 -
Flags: review?(mschroeder) → review+
Updated•13 years ago
|
Keywords: checkin-needed
Comment 4•13 years ago
|
||
Checked in: http://hg.mozilla.org/comm-central/rev/1155c1c0ed0d
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 1.0
Comment 5•11 years ago
|
||
These bugs are likely targeted at Lightning 1.0b1, not Lightning 1.0. If this change was done in error, please adjust the target milestone to its correct value. To filter on this bugspam, you can use "lightning-10-target-move".
Target Milestone: 1.0 → 1.0b1
You need to log in
before you can comment on or make changes to this bug.
Description
•