Opened 12 years ago

Closed 12 years ago

#2072 closed defect (fixed)

the changelists.css file references two styles which are not defined : _height & _margin-right

Reported by: darren.redmond@… Owned by: Wilson Miner
Component: Generic views Version:
Severity: normal Keywords: css stylesheet
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


This relates to the trunk checked out yesterday June 1st at 14:32 BST.

The changelists.css stylesheet was not displaying correctly on Apache server 2.0.58 viewed via Firefox, and there were two style sheet errors in the java script console relating to line 7 (_height) and line 30 (_margin-right).

When I changed the references from _height to height and from _margin-right to margin-right, the ui now displays correctly.

existing line 7 changelists.css:
.change-list .filtered { min-height:400px; _height:400px; }

changed line 7 changelists.css:
.change-list .filtered { min-height:400px; height:400px; }

existing line 30 changelists.css:
#changelist-filter ul { padding-left:0;margin-left:10px;_margin-right:-10px; }

changed line 30:
#changelist-filter ul { padding-left:0;margin-left:10px;margin-right:-10px; }


Change History (2)

comment:1 Changed 12 years ago by Wilson Miner

Owner: changed from Jacob to Wilson Miner
Status: newassigned

Those are intentional rules to fix rendering bugs in IE. Only IE (6 and earlier) interprets rules preceded by an underscore. I'll move them out to the patch-iewin.css file to avoid further confusion.

comment:2 Changed 12 years ago by Wilson Miner

Resolution: fixed
Status: assignedclosed

(In [3064]) Fixed #2072 -- Moved IE hacks out of changelists.css into patch-iewin.css to avoid confusion.

Note: See TracTickets for help on using tickets.
Back to Top