Opened 7 years ago

Closed 6 years ago

Last modified 6 years ago

#15078 closed bug (fixed)

Selectmenu: calling refresh repeatedly makes it slower and slower

Reported by: henkpb Owned by: Scott González
Priority: minor Milestone: 1.12.2
Component: ui.selectmenu Version: 1.12.1
Keywords: Cc:
Blocked by: Blocking:

Description

Following up on case #15049:

I have encountered a similar problem in my application, where selectmenu("refresh") gets slower and slower. Please check http://jsbin.com/nibibohigi/edit?html,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.

Change History (3)

comment:1 Changed 6 years ago by Evan Easton

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.

comment:2 Changed 6 years ago by Scott González

Owner: set to Scott González
Resolution: fixed
Status: newclosed

In ef2e9ba:

Widget: Improve remove event bindings for classes options

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

comment:3 Changed 6 years ago by Scott González

Milestone: none1.12.2
Note: See TracTickets for help on using tickets.