Skip to main content

Search and Top Navigation

#15078 closed bug (fixed)

Opened October 12, 2016 05:00PM UTC

Closed April 19, 2017 04:54PM UTC

Last modified April 19, 2017 04:56PM UTC

Selectmenu: calling refresh repeatedly makes it slower and slower

Reported by: henkpb Owned by: scottgonzalez
Priority: minor Milestone: 1.12.2
Component: ui.selectmenu Version: 1.12.1
Keywords: Cc:
Blocked by: Blocking:

Following up on case #15049:

I have encountered a similar problem in my application, where selectmenu("refresh") gets slower and slower. Please check,js,output for a basic example.

I have seen that the

this.bindings = this.bindings.add( element ) 

in the _on method in $.Widget.prototype causes the problem. The list of bindings increases on each refresh - it ends up containing thousands of entries.

Attachments (0)
Change History (3)

Changed February 20, 2017 06:42PM UTC by Evaneaston comment:1

We're getting burned by this too. We've got to do .val(...).selectmenu('refresh') to sync with our ractive model...and now if you arrow through a long list of options, it starts getting slower and slower for the reason described in the description.

Changed April 19, 2017 04:54PM UTC by scottgonzalez comment:2

owner: → scottgonzalez
resolution: → fixed
status: newclosed

In [changeset:"ef2e9bab92ae898311baa295590cd487d9071319" ef2e9ba]:

#!CommitTicketReference repository="" revision="ef2e9bab92ae898311baa295590cd487d9071319"
Widget: Improve  event bindings for  options

Fixes #15078
Fixes #15082
Fixes #15095
Fixes #15136
Fixes #15152
Closes gh-1769

Changed April 19, 2017 04:56PM UTC by scottgonzalez comment:3

milestone: none1.12.2