Search and Top Navigation
#7716 closed bug (fixed)
Opened September 13, 2011 06:12PM UTC
Closed November 16, 2011 03:52PM UTC
Last modified November 16, 2011 03:53PM UTC
Selectable: new elements added programatically don't have class ui-selectee
Reported by: | millermedeiros | Owned by: | rdworth |
---|---|---|---|
Priority: | minor | Milestone: | 1.8.17 |
Component: | ui.selectable | Version: | 1.8.16 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
adding new elements to the selectable list using append
and appendTo
won't update class names, causing styling conflicts.
here is a working example and more info: http://jsfiddle.net/Lhgua/
Attachments (0)
Change History (5)
Changed November 11, 2011 08:36PM UTC by comment:1
Changed November 16, 2011 03:51PM UTC by comment:2
owner: | → rdworth |
---|---|
status: | new → assigned |
summary: | new elements added programatically don't have class ui-selectee → Selectable: new elements added programatically don't have class ui-selectee |
Changed November 16, 2011 03:52PM UTC by comment:3
resolution: | → fixed |
---|---|
status: | assigned → closed |
Selectable: add ui-selectee class to elements matching filter on refresh. Fixes #7716 - Selectable: new elements added programatically don't have class ui-selectee
Changeset: 97bc2d9de9875e222254cfa1cd3fa3bde605c2ed
Changed November 16, 2011 03:53PM UTC by comment:4
Selectable: add ui-selectee class to elements matching filter on refresh. Fixes #7716 - Selectable: new elements added programatically don't have class ui-selectee
(cherry picked from commit 97bc2d9de9875e222254cfa1cd3fa3bde605c2ed)
Changeset: e9002c4c8f04266497926419034ea44f62725bcf
Changed November 16, 2011 03:53PM UTC by comment:5
milestone: | 1.9 → 1.8.17 |
---|
One fix is to put '$this.addClass("ui-selectee");' inside the refresh method, and remove 'this.selectees = selectees.addClass("ui-selectee");' from the '_create' method.
@millermedeiros
Whatever the fix will be, you'll have to call 'refresh' since 'append' method is a jQuery core method, and not part of jQuery UI (unless than jQuery UI override this method or use the MutationEvent to do more work)