Skip to content

Conversation

NickeZ
Copy link
Collaborator

@NickeZ NickeZ commented Oct 1, 2025

No description provided.

@NickeZ NickeZ requested a review from benma October 1, 2025 09:35
@NickeZ NickeZ force-pushed the fix/bottom-tap-while-gesture-confirm branch 3 times, most recently from 2fa485b to f271546 Compare October 1, 2025 10:21
Comment on lines 244 to 245
// Render confirm button or gesture (render confirm gesture last so that bottom triangle is
// above keyboard)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so that bottom triangle is above keyboard

The keyboard is not rendered while the confirm gesture is active. Is this comment about something else?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not true, you can clearly see the bottom triangle "animating out" beneath the keyboard.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So the keyboard (and everything else) is not rendered while the bottom triangle is animating in, but it is rendered when it is animated out. Not a big deal, but I wonder if that should be changed so it waits until the bottom triangle is gone before rendering the rest again.

Quick patch to achieve this (not sure if this is a good way, but it works):

Screenshot_2025-10-13_09-49-12

Feels pretty good in my opinion, please try it.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also tried it, I really like it. Switched to that implementation.

Extend the duration for when the confirm gesture is active so that
involuntary bottom taps are not triggered and so that the confirm
gesture triangle isn't rendered blended into the keyboard.
@NickeZ NickeZ force-pushed the fix/bottom-tap-while-gesture-confirm branch from f271546 to b586290 Compare October 13, 2025 09:10
@NickeZ NickeZ requested a review from benma October 13, 2025 09:12
Copy link
Collaborator

@benma benma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK

@NickeZ NickeZ merged commit 77af14c into BitBoxSwiss:master Oct 13, 2025
32 checks passed
@NickeZ NickeZ deleted the fix/bottom-tap-while-gesture-confirm branch October 13, 2025 10:36
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.

2 participants