Changes between Initial Version and Version 1 of Ticket #10050, comment 10


Ignore:
Timestamp:
Jan 2, 2018, 8:17:23 AM (6 years ago)
Author:
Ryan J Ollos
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #10050, comment 10

    initial v1  
    1 Thanks indeed, the workaround works a treat.  Would this not be suitable to be included in the jquery ui source?
    2 
    31Replying to [comment:9 sklarej]:
    42> I can confirm that there is an issue with some kind of leak that has performance implications as of jQuery UI 1.12.1 / jQuery 3.2.1.
    5 >
    6 > You can easily duplicate this using the jQuery Combobox demo by doing the following:
    7 >
    8 > 1. Goto the jQuery UI Autocomplete Combobox demo at [https://jqueryui.com/autocomplete/#combobox]
    9 > 2. Repeatedly press the dropdown menu arrow on the right of the combobox to show all items
    10 > 3. After about 20 times (this is dependent on your computer speed) you will start to see a slowdown in the time it takes to display the dropdown list.
    11 > 4. Every time you click the dropdown, it will take longer to display
    12 > 5. Autocomplete speed is also compromised
    13 >
    14 > While this is not very noticeable with a smaller list of items, it can severely affect the user experience with a larger list.
    15 >
    16 > We have some comboboxes with a large number of items (up to ~500 items).  The autocomplete works quickly enough at first, but the more you type in the slower it goes.  Even if you just open a jquery combobox dropdown list repeatedly, it gets slower and slower each time.  We've had reports of typing in the contents of an autocomplete field taking over a minute.
    17 >
    18 > '''Workaround''': The workaround described in jGeek314's comment about clearing out the menu bindings had allowed us to continue to use the autocomplete control:
    19 >
    20 > {{{
    21 > autoComplete.data("ui-autocomplete").menu.bindings = $();
    22 > }}}
    23 >
     3> ...
     4
     5
     6Thanks indeed, the workaround works a treat.  Would this not be suitable to be included in the jquery ui source?