Opened 8 years ago
Closed 8 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 8 years ago by
Attachment: | test_trac15817.html added |
---|
comment:1 Changed 8 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 tablets, thus forcing a browser scroll; 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 8 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 8 years ago by
Attachment: | patch15817.patch added |
---|
Getting rid of dojo/touch from dojox/mobile/_ItemBase - Adrian Vasiliu, IBM, CCLA
comment:4 Changed 8 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 8 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 8 years ago by
Milestone: | tbd → 1.8 |
---|---|
Resolution: | → fixed |
Status: | new → closed |
I fixed #15821 so this is fixed too.
Test case to reproduce.