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: |
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.
Attachments (0)
Change History (3)
Changed February 20, 2017 06:42PM UTC by comment:1
Changed April 19, 2017 04:54PM UTC by comment:2
| owner: | → scottgonzalez |
|---|---|
| resolution: | → fixed |
| status: | new → closed |
In [changeset:"ef2e9bab92ae898311baa295590cd487d9071319" ef2e9ba]:
#!CommitTicketReference repository="" revision="ef2e9bab92ae898311baa295590cd487d9071319" Widget: Improveevent bindings foroptions Fixes #15078 Fixes #15082 Fixes #15095 Fixes #15136 Fixes #15152 Closes gh-1769
Changed April 19, 2017 04:56PM UTC by comment:3
| milestone: | none → 1.12.2 |
|---|
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.