Opened 11 years ago
Closed 11 years ago
#11111 closed defect (fixed)
[regression] IE60 shows an extra margin for ContentPanes if using the GridContainer's CSS
Reported by: | Martin Jakobi | Owned by: | Shane O'Sullivan |
---|---|---|---|
Priority: | high | Milestone: | 1.6 |
Component: | Dojox | Version: | 1.5.0b2 |
Keywords: | css dijit ContentPane ie60 | Cc: | jfcunat |
Blocked By: | Blocking: |
Description (last modified by )
All ContentPanes get an extra margin in the IE60 if the GridContainer's CSS (dojo/dojox/layout/resources/GridContainer.css) is used.
That CSS contains starting at line 68:
.dj_ie6 .dijitContentPane, .dj_ie6 .dojoxPortlet, .dj_ie6 .dijitTitlePane{ margin: 10px; }
Removing line 68 or not using the css the ContentPanes are rendered as expected beside the GridContainer.
See the testcase with a GridContainer? on the "second page". With removing line 12 in the test file the page is rendered as expected.
Attachments (1)
Change History (5)
Changed 11 years ago by
Attachment: | tcExtraMarginOnIE60.html added |
---|
comment:1 Changed 11 years ago by
Cc: | jfcunat added |
---|---|
Component: | Dijit → Dojox |
Description: | modified (diff) |
Milestone: | tbd → 1.5 |
Owner: | set to Shane O'Sullivan |
Summary: | IE60 shows an extra margin for ContentPanes if using the GridContainer's CSS → [regression] IE60 shows an extra margin for ContentPanes if using the GridContainer's CSS |
I agree, that CSS is weird, GridContainer?.css shouldn't have rules that affect ContentPane, except maybe if the ContentPane is inside a GridContainer. Still, I don't see why there's a special rule for IE6.
That code was added as part of the new GridContainer, in [21950], for #10680.
comment:2 Changed 11 years ago by
Milestone: | 1.5 → 1.6 |
---|
comment:3 Changed 11 years ago by
(In [22753]) Refs #11111 Changes the IE6 CSS rules to be more specific, so that only ContentPane? and TitlePane? widget that are in Portlets are affected, rather than all widgets on the page
comment:4 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
testcase