-
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
[lexical-playground] Feature: Add touch support for TableCellResizer #7299
[lexical-playground] Feature: Add touch support for TableCellResizer #7299
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
This all seems to work well for me on macOS Chrome, iOS Safari and Samsung Galaxy S23 Chrome in browserstack. Nice job!
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.
Seems to still work well on macOS Chrome, Safari and Firefox as well as iOS Safari (hardware) and Chrome on Samsung Galaxy S23 (browserstack). Nice job!
This pull request includes several changes to improve touch support for the
TableCellResizer
plugin.Enhancements to touch support and event handling:
packages/lexical-playground/src/plugins/TableCellResizer/index.css
: Added styles for coarse pointer devices to improve touch interaction.packages/lexical-playground/src/plugins/TableCellResizer/index.tsx
: ReplacedMouseEvent
withPointerEvent
to handle both mouse and touch events, and added specific handling for touch events such asonTouchMove
andonClick
. [1] [2] [3] [4] [5] [6] [7]General improvements:
packages/lexical-playground/src/plugins/TableCellResizer/index.tsx
: Updated thezoneWidth
and adjusted the calculation of resizer styles for better accuracy and usability.