-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Feat (Whiteboards): Pen mode (palm rejection) #10222
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On Galaxy Tab S7+ with S Pen:
when "Pen mode" is on, even the tip of the pen is not touching the screen, a line will be drawn. It also connects directly to the tail of the last line.
S Pen has a special mode, when the tip is close to the screen (about 2cm), a transparent cursor will be shown on the screen.
@andelf Sorry for the late reply.
I don't have a samsung device to test this, but I tried to fix this blindly using this piece of information. I assume that this is a secondary pointer, so checking for isPrimary might fix this. Let me know if that improved the situation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on Galaxy Tab S7+. LGTM
c7692e0
to
aed0dc1
Compare
A simplified approach to pen mode. When this mode is enabled, non 'pen' type pointer events will be rejected.
Although this feature needs to be explicitly enabled, it would be nice if we could test this on multiple different devices before release. It seems to work fine on my wacom tablet on linux. I also made some minor style adjustments to our bottom toolbars for mobile.
Resolves #9863
Also see
https://discuss.logseq.com/t/whiteboards-could-really-do-with-palm-rejection-for-use-on-tablets/16769
https://discuss.logseq.com/t/whiteboard-ipad-apple-pencil-support/17526