Search and Top Navigation
#3331 closed bug (worksforme)
Opened September 10, 2008 04:39PM UTC
Closed September 18, 2008 10:12AM UTC
Last modified October 11, 2012 09:15PM UTC
Injection of span elements screws up list in IE7
| Reported by: | noah@computerinnovationsny.com | Owned by: | jzaefferer |
|---|---|---|---|
| Priority: | major | Milestone: | |
| Component: | ui.accordion | Version: | 1.5.2 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
I've modified my ui.accordion.js file to read:
if (!this.options.disableLeftSpan)
$("<span class='ui-accordion-left'/>").insertBefore(options.headers);
if (!this.options.disableRightSpan)
$("<span class='ui-accordion-right'/>").appendTo(options.headers);
on lines 42-45 to fix an issue whereby the span elements that where injected before and after my list items were adding tons of white-space above the list items. Maybe this only affected me because I have a heavily styled <ul> with <li>s using background images, but whatever caused it, this really broke the design bigtime. Now, with my edits, I can just specify {disableRightSpan: true, disableLeftSpan: true} to prevent this.
You can achieve nearly the same effect by adding the ui-accordion class to your container element. The only thing missing will be tab-index and ui-accordion-header classes on header elements, but you don't really need them when not using a UI theme anyway.