Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#3844 closed bug (fixed)

Accordion icons are not being shown correctly in IE(6)

Reported by: eduardo Owned by:
Priority: blocker Milestone: 1.7
Component: ui.accordion Version: 1.6rc5
Keywords: Cc:
Blocked by: Blocking:

Description (last modified by rdworth)

In the default accordion in IE6, the icons display way below the accordion. This is not a demos issue, as it can be seen on an individual static and visual test pages

http://jquery-ui.googlecode.com/svn/trunk/tests/static/accordion.html

http://jquery-ui.googlecode.com/svn/trunk/tests/visual/accordion.html

Change History (8)

comment:1 Changed 8 years ago by eduardo

  • Component changed from ui.core to ui.accordion

comment:2 Changed 8 years ago by scottgonzalez

  • Milestone changed from TBD to 1.6
  • Priority changed from blocker to critical

comment:3 Changed 8 years ago by paul

  • Component changed from ui.accordion to [meta] demos
  • Summary changed from Accordion icons are not being shown on IE7 to Accordion icons are not being shown on IE7 on demos/index.html

Not a bug in in Accordion, but in the demos/index.html.

comment:4 Changed 8 years ago by rdworth

  • Component changed from [meta] demos to ui.accordion
  • Description modified (diff)
  • Priority changed from critical to blocker
  • Summary changed from Accordion icons are not being shown on IE7 on demos/index.html to Accordion icons are not being shown correctly in IE(6)

comment:5 Changed 8 years ago by jzaefferer

The culprit is this definition in ui.accordion.css:

.ui-accordion .ui-icon { position: absolute; left: .5em; top: 50%; margin-top: -8px; }

More specifically, the top: 50% throws IE off, for whatever reason. Not sure yet how to workaround that.

comment:6 Changed 8 years ago by jzaefferer

  • Resolution set to fixed
  • Status changed from new to closed

Replacing top: 50% with top: 1.15em seems to fix the issue in IE6 while keeping other browsers intact as well.

Fixed in r1724.

comment:7 follow-up: Changed 8 years ago by bohdan.ganicky

I modified the fix. The issue is that IE6 is able to position to a parent with triggered hasLayout only. You can trigger the hasLayout with simply defining height: 1% or zoom: 1 for IE6 only.

comment:8 in reply to: ↑ 7 Changed 8 years ago by bohdan.ganicky

Modified fix in r1725.

Note: See TracTickets for help on using tickets.