Opened 3 years ago
Closed 3 years ago
#13348 closed defect (bug) (fixed)
IE 7 nav menu admin styling broken
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | high | Milestone: | 3.0 |
| Component: | Menus | Version: | 3.0 |
| Severity: | blocker | Keywords: | has-patch |
| Cc: |
Description
There are some issues under IE 7.
Attachments (4)
Change History (16)
I don't think you should be adding those unsemantic "clear" divs, especially as the clearfix CSS hack has no effect on IE anyways.
Fixes for IE6 & 7 CSS should go to ie.css which is conditionally loaded, no need for css hacks in the main stylesheet.
Replying to azaozz:
Fixes for IE6 & 7 CSS should go to ie.css which is conditionally loaded, no need for css hacks in the main stylesheet.
For IE 6 yes, but not for IE 7. <!--[if lte IE 7]> is only for IE 6.
I added a new patch without unsemantic "clear" divs. Only CSS and one fix in nav-menu.php.
- Cc has-patch removed
- Keywords needs-patch added; has-patch removed
"lte IE7" means less than or equal. ie.css gets served for both 6 and 7.
I would want koopersmith, maybe filosofo to review anything going into the main stylesheet to ensure we're not breaking anything else.
comment:10
nacin — 3 years ago
- Owner changed from ocean90 to koopersmith
- Status changed from accepted to assigned
comment:11
filosofo — 3 years ago
With my patch on #13388, IE 7 seems pretty good too.

use IE 7 selector instead of JS detection