Skip to content

feat/close menu by reverse gesture#297

Open
wassupluke wants to merge 10 commits into
jrpie:masterfrom
wassupluke:feat/close-menu-by-reverse-gesture
Open

feat/close menu by reverse gesture#297
wassupluke wants to merge 10 commits into
jrpie:masterfrom
wassupluke:feat/close-menu-by-reverse-gesture

Conversation

@wassupluke
Copy link
Copy Markdown
Contributor

To close #174

I've stayed up way too long looking at this problem. Too tired to pretty this up with markdown at the moment. Here's my work in action: https://youtube.com/shorts/ExDc7Z_frJo?feature=share

Closes the app menu by swipe_dn from 1) the menu, 2) the All Apps title, or 3) the "x". I left the settings icon alone so swiping down on it should not close the app list, as I figured that could be a frustration point for users trying to open settings but accidentally put a little swipe_dn motion on their tap.

Doesn't close the list if scrolling a bit and then back to/past the start in the same gesture. Doesn't close the list if you scroll with great momentum (fling). I swear there was one other edge case I caught and handled but I didn't write it down.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feature] Close menus by reverse gesture (for straight-line gestures)

1 participant