[PlayBook] Context menu should not appear when scrolling a list of items #370

Closed
tneil opened this Issue Oct 4, 2012 · 4 comments

Comments

Projects
None yet
2 participants
@tneil
Member

tneil commented Oct 4, 2012

Currently when you are scrolling a list of items and your finger stays on the list item for a "press and hold" it is popping up the context menu.

The context menu should only appear if you are not in the process of scrolling.

@tneil tneil referenced this issue Oct 4, 2012

Closed

peek context #355

@tneil

This comment has been minimized.

Show comment Hide comment
@tneil

tneil Dec 11, 2012

Member

I believe this could be resolved on BB10 with the resolution to issue #233. It would however likely remain an issue for PlayBook until it receives BB10 as an upgrade

Member

tneil commented Dec 11, 2012

I believe this could be resolved on BB10 with the resolution to issue #233. It would however likely remain an issue for PlayBook until it receives BB10 as an upgrade

@dirkjanm

This comment has been minimized.

Show comment Hide comment
@dirkjanm

dirkjanm Jan 9, 2013

Contributor

This can relatively easy be done by adding the current X scroll position to the itemNode.ontouchstart function, then compare that to the current X scroll position when the touchTimer function triggers, if it is more than 50 px difference, disregard it because the user is scrolling.
I've sort of hacked this into the BBui.js version I'm currently working with, can send you that but I dont feel like spending a lot of time on filling in and signing all kinds of forms.

Contributor

dirkjanm commented Jan 9, 2013

This can relatively easy be done by adding the current X scroll position to the itemNode.ontouchstart function, then compare that to the current X scroll position when the touchTimer function triggers, if it is more than 50 px difference, disregard it because the user is scrolling.
I've sort of hacked this into the BBui.js version I'm currently working with, can send you that but I dont feel like spending a lot of time on filling in and signing all kinds of forms.

@tneil

This comment has been minimized.

Show comment Hide comment
@tneil

tneil Jan 9, 2013

Member

Yup.. I was thinking the same thing. I'll likely only apply this to the PlayBook logic though. We will be transitioning the bbUI context menu for BB10 to use the underlying webworks menu implementation which doesn't have this issue.

Member

tneil commented Jan 9, 2013

Yup.. I was thinking the same thing. I'll likely only apply this to the PlayBook logic though. We will be transitioning the bbUI context menu for BB10 to use the underlying webworks menu implementation which doesn't have this issue.

@tneil

This comment has been minimized.

Show comment Hide comment
@tneil

tneil Feb 25, 2013

Member

This should now be fixed

Member

tneil commented Feb 25, 2013

This should now be fixed

@tneil tneil closed this Feb 25, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment