#7396 closed feature (invalid)

Opened May 19, 2011 12:03AM UTC

Closed November 08, 2012 08:53AM UTC

Selectable _create() slow on IE6 with large number of selectees

Reported by: lachlank Owned by: lachlank
Priority: minor Milestone: 2.0.0
Component: ui.selectable Version: 1.8.13
Keywords: Cc:
Blocked by: Blocking:
Description

Due to the slowness of offset() under IE6 (and probably 7 & 8), calling selectable() with a large number of selectees can take several seconds. If the autoRefresh option is specified as true, the initial call to refresh() in _create() is probably unnecessary?

More specifically, line 38:

var pos = $this.offset();

could be rewritten as:

var pos = first && autoRefresh ? {left: 0, top: 0} : $this.offset();
first = false;

where first and autoRefresh are defined in the body of _create:

var first = true;
var autoRefresh = this.options.autoRefresh();

This is probably overly simplistic, but works in my situation.

I am working in a corporate environment running IE6 so unfortunately these performance issues that are probably not noticeable on modern browsers can be a major headache.

Attachments (0)
Change History (5)

Changed November 12, 2011 01:15PM UTC by ghostd comment:1

https://github.com/jquery/jquery-ui/pull/524

Changed June 26, 2012 01:22AM UTC by scottgonzalez comment:2

type: enhancement → feature

Changed October 11, 2012 02:52PM UTC by scottgonzalez comment:3

milestone: 1.9.0 → 2.0.0

Changed October 24, 2012 08:29PM UTC by petersendidit comment:4

owner: → lachlank
status: new → pending

Thanks for taking the time to contribute to the jQuery UI project! Please provide a complete reduced test case on jsFiddle to help us assess your ticket. We are dropping support for IE6 in jQuery UI 1.10, but if you can create an example of it being slow in IE7/8 that would help.

Additionally, be sure to test against the git version of both jQuery UI and jQuery to ensure the issue still exists. To get you started, use this boilerplate: http://jsfiddle.net/ZgAqH/Open the link and click to "Fork" (in the top menu) to get started.

Changed November 08, 2012 08:53AM UTC by trac-o-bot comment:5

resolution: → invalid
status: pending → closed

Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!