Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Back gesture is recognized as drawing intent #80

Merged
merged 1 commit into from
Nov 11, 2024

Conversation

tuancoltech
Copy link
Collaborator

@tuancoltech tuancoltech commented Oct 18, 2024

Problem

When back gesture navigation is made from either left or right side of screen, it's mistakenly recognized as a drawing intent
in graphics note edit mode. This affects badly on UX.

https://app.asana.com/0/1207819682524134/1208536199909620

Solution

Do not draw if the gesture touch point is made within the edge (either left or right) side of screen.

Before

Screen.Recording.2024-10-18.at.21.42.15.mov

After

Screen.Recording.2024-10-18.at.21.47.18.mov

@tuancoltech tuancoltech added the bug Something isn't working label Oct 18, 2024
@tuancoltech tuancoltech self-assigned this Oct 19, 2024
@tuancoltech tuancoltech force-pushed the feat/implement_graphics_note_thumb branch from db285fd to fcc0550 Compare October 27, 2024 09:19
@tuancoltech tuancoltech changed the base branch from feat/implement_graphics_note_thumb to main October 27, 2024 09:21
@tuancoltech tuancoltech force-pushed the enhance/back_gesture_conflict_draw_intent branch from e44521d to ab6e6ee Compare October 27, 2024 09:23
@tuancoltech tuancoltech changed the base branch from main to feat/implement_graphics_note_thumb October 27, 2024 09:25
@tuancoltech tuancoltech force-pushed the feat/implement_graphics_note_thumb branch from fcc0550 to ea4ca86 Compare November 7, 2024 05:49
@tuancoltech tuancoltech changed the base branch from feat/implement_graphics_note_thumb to main November 7, 2024 15:26
@tuancoltech tuancoltech force-pushed the enhance/back_gesture_conflict_draw_intent branch from ab6e6ee to 86ee76b Compare November 7, 2024 15:30
@tuancoltech tuancoltech merged commit bcb614e into main Nov 11, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants