Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Listview with initially hidden items breaks when refreshed - jQM 1.3.0 #5680

Closed
GodsBest opened this Issue · 7 comments
@GodsBest

In jQM 1.3.0 if a listview with initially hidden items is updated by adding new items and then calling .listview( 'refresh' ), the listview breaks. The newly added items do not hide.
http://jsfiddle.net/fiddleyetu/ffMhk/2/
If you comment out the last two lines of the javascript and click run, you'll see that it works as expected.

However, if the entire listview (including ul or ol element) is dynamically created by putting the dynamically generated listview markup in a container element and then calling .trigger( 'create' ) on the container element. it works fine.
http://jsfiddle.net/fiddleyetu/ffMhk/

@jaspermdegroot
Collaborator

Test page that uses latest code: http://jsbin.com/edeluh/68
This is something for the listview widget review.

@gabrielschulhof gabrielschulhof self-assigned this
@gabrielschulhof
Collaborator

Actually, I would argue that, like, #6911, this is part of providing correct backwards compatibility with 1.3.0, and therefore the milestone should be 1.4.x.

@gabrielschulhof
Collaborator

OK, I misunderstood. This never worked in the first place. Still, since it's a bug in 1.4.x, we should fix it. The fix does not require any refactoring. It's simply a bug fix.

@gabrielschulhof gabrielschulhof modified the milestone: 1.4.3, 1.5.0
@GodsBest

Just to be sure we're on the same page issue #5680 remains unresolved to date:
http://jsbin.com/edeluh/68
http://jsfiddle.net/fiddleyetu/4gWXt/show/

What's the timeline for resolving it?

@jaspermdegroot
Collaborator

@gabrielschulhof - #7339 landed in 1.4-stable, right? Can we can close this ticket as fixed?

@jaspermdegroot jaspermdegroot added the Fixed label
@gabrielschulhof
Collaborator

Yeah, we can close it.

@gabrielschulhof
Collaborator

Fixed in baa6932

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.