Skip to main content

Search and Top Navigation

#5413 closed bug (fixed)

Opened March 24, 2010 04:14AM UTC

Closed February 11, 2012 11:30PM UTC

Last modified February 11, 2012 11:32PM UTC

Accordion + Sortable doesn't work properly on IE6/7

Reported by: oxez Owned by:
Priority: minor Milestone: 1.8.18
Component: ui.accordion Version: 1.8
Keywords: accordion sortable ie7 Cc:
Blocked by: Blocking:
Description

When trying out this demo: http://jqueryui.com/demos/accordion/#sortable on IE7, sorting the elements result in them disappearing. I tried looking for a workaround but couldn't find anything.

Attachments (0)
Change History (9)

Changed March 26, 2010 05:41AM UTC by oxez comment:1

A workaround I've found is to use:

$("body").css("display", "none");
$("body").css("display", "block");

However it's a dirty workaround... but that's as much as I can do to help.

Changed July 13, 2010 01:46PM UTC by scottgonzalez comment:2

summary: Accordion + Sortable doesn't work properly on IE7Accordion + Sortable doesn't work properly on IE6/7

Changed July 14, 2010 06:29PM UTC by scottgonzalez comment:3

milestone: TBD1.next

Changed December 03, 2010 12:23AM UTC by mikeschuld comment:4

_comment0: The demo link posted above does not break anymore. The demo in the svn trunk DOES break this way. \ \ http://jquery-ui.googlecode.com/svn/trunk/demos/accordion/sortable.html \ \ Attempting to cause the same problem in JSFiddle does not reproduce the issue though.1291336394602795

The demo in the svn trunk also breaks this way.

http://jquery-ui.googlecode.com/svn/trunk/demos/accordion/sortable.html

Attempting to cause the same problem in JSFiddle does not reproduce the issue though.

Changed December 03, 2010 01:21AM UTC by mikeschuld comment:5

This seems to be due to the hasLayout IE uses. Got a workaround fix by alternating the zoom on the handle of the accordion header to 0 and 1.

// outsize the accordion (global)
var zoom = 1;

// in .sortable
stop: function {
    zoom = zoom ? 0 : 1;
    setTimeout('$(".ui-accordion-header").css("zoom", ' + zoom + ')', 10);
}

I can work on a fix in the jquery source itself but I have not ever done so, so if someone more experienced within the git and everything would rather do it for me feel free.

This has only been tested by me in IE7

Changed March 28, 2011 08:55PM UTC by scottgonzalez comment:6

status: newopen

Changed February 11, 2012 11:30PM UTC by Scott González comment:7

resolution: → fixed
status: openclosed

Accordion demo: Work around some IE quirks in the sortable demo. Fixes #5413 - Accordion + Sortable doesn't work properly on IE6/7.

Changeset: 00b69db88365e41afbed29b9fed3592e0703b8ed

Changed February 11, 2012 11:31PM UTC by Scott González comment:8

Accordion demo: Work around some IE quirks in the sortable demo. Fixes #5413 - Accordion + Sortable doesn't work properly on IE6/7.

(cherry picked from commit 00b69db88365e41afbed29b9fed3592e0703b8ed)

Changeset: e77249eb3e06064d1e8c09a41047fb644818eb3a

Changed February 11, 2012 11:32PM UTC by scottgonzalez comment:9

milestone: 1.next1.8.18