Search and Top Navigation
#7258 closed enhancement (fixed)
Opened April 15, 2011 03:11PM UTC
Closed May 02, 2011 10:03PM UTC
Last modified July 30, 2011 11:57AM UTC
Dialog: optimize initialization
Reported by: | adambaratz | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 1.9.0 |
Component: | ui.dialog | Version: | 1.8.11 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
Assorted changes to speed up loading:
- _create: Delay appending the container div to the body until the end of the function. Since more manipulations are going to occur between the creation of that div and the end of that function, it's adding work for the browser to append it at an early stage.
- open: There's currently a chain of concat calls to find the first tabbable element. Since the first query may return a match, it's wasteful to keep looking if there was a match.
- _createButtons: A couple divs are created, but they're not necessarily used. Check whether there are any buttons before doing the work. Returning any resulting div facilitates "subclassing" this plugin.
Attachments (0)
Change History (3)
Changed April 15, 2011 03:14PM UTC by comment:1
Changed May 02, 2011 10:03PM UTC by comment:2
resolution: | → fixed |
---|---|
status: | new → closed |
Dialog: remove unneeded DOM manipulations. Fixed #7258 - optimize initialization.
Changeset: 51df02ee4ea02e91919842a169463612176682d5
Changed July 30, 2011 11:57AM UTC by comment:3
This stil performs poorly with the latest version (1.8.14) and a new dialog with arount 300 elements (Firefox 5 takes around 5 seconds)
https://github.com/adambaratz/jquery-ui/commit/dfb7f3982855e55827c9f8aa80e7419261fbca2c