Skip to content

fix: disabled split gutter interactions #502

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

Conversation

Harpush
Copy link
Collaborator

@Harpush Harpush commented Feb 3, 2025

Chrome fires both mousedown and touchstart events. When split is disabled there was no prevent default which resulted in two click events firing. This means that clicking on a disabled gutter and changing size will not work (see https://angular-split.github.io/examples/gutter-click-roll-unroll in mobile mode using chrome devtools and clicking the gutter)

@Harpush Harpush requested a review from Jefiozie February 3, 2025 23:26
@Harpush Harpush force-pushed the fix-touch-devices-gutter-interaction-when-disabled branch from a3daeee to d19c860 Compare February 4, 2025 06:11
@Jefiozie Jefiozie merged commit 520c5f8 into angular-split:main Mar 12, 2025
7 checks passed
@Harpush Harpush deleted the fix-touch-devices-gutter-interaction-when-disabled branch May 30, 2025 16:20
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