Opened 10 years ago
Closed 10 years ago
#15817 closed defect (fixed)
ListItem transitions when scrolling
Reported by: | kbenjamin | Owned by: | Eric Durocher |
---|---|---|---|
Priority: | undecided | Milestone: | 1.8 |
Component: | DojoX Mobile | Version: | 1.8.0rc1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
Actually, 1.8rc2 but there isn't a choice for that.
When you have more than a screenful of ListItem? widgets and try to touch scroll through them they often fire the onClick event i.e. moveTo, instead of only scrolling.
Attachments (2)
Change History (8)
Changed 10 years ago by
Attachment: | test_trac15817.html added |
---|
comment:1 Changed 10 years ago by
The attached test_trac15817.html is a modified version of dojox/mobile/tests/test_ListItem-actions.html. The differences: increased number of items in the first list such that they don't fit in the screen height on either phones or tablettes; also, the transitions are based on "moveTo" instead of onclick actions (however the bug hurts in both cases).
How to reproduce: load the file on a mobile device and scroll a couple of times. After only a few scrolling gestures, a view transition is triggered, while it shouldn't.
I reproduce with iPhone 4S iOS 5.0.1, iPad 2 iOS 5.1.1, Galaxy Tab Android 2.2. Apparently hurts all mobile devices. Not reproducible on desktop browsers (specific to touch-enabled devices).
Appears to be a side-effect of changes in dojo/touch.js. In any case, reverting touch.js to rev. 26731 (5 months old), the issue is NOT reproducible anymore. We will see how to deal with it.
comment:2 Changed 10 years ago by
Update:
More exactly, the trouble is since rev. 28381 of dojo/touch.js. This will be reported separately (as a dojo/touch issue; a symptom can be reproduced using dojo/tests/test_touch.html).
On our side, we will probably go for getting rid of using dojo/touch in _ItemBase.
Changed 10 years ago by
Attachment: | patch15817.patch added |
---|
Getting rid of dojo/touch from dojox/mobile/_ItemBase - Adrian Vasiliu, IBM, CCLA
comment:4 Changed 10 years ago by
The attached patch15817.patch gets rid of the use of dojo/touch in dojox/mobile/_ItemBase. This fixes the trouble reported in this ticket. As agreed with edurocher, we may use this solution in case we need a quick workaround waiting for the fix in dojo/touch.
comment:5 Changed 10 years ago by
By the way, since the bug occurs only when native browser scrolling is used, a quick and easy workaround is to replace the toplevel View by a ScrollableView.
comment:6 Changed 10 years ago by
Milestone: | tbd → 1.8 |
---|---|
Resolution: | → fixed |
Status: | new → closed |
I fixed #15821 so this is fixed too.
Test case to reproduce.