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.

Sign up for GitHub

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

Jump to bottom

Switch to pointer events and remove custom event handling logic #6543

Closed
maribethb opened this issue Oct 13, 2022 · 1 comment
Closed

Switch to pointer events and remove custom event handling logic #6543

maribethb opened this issue Oct 13, 2022 · 1 comment
Assignees
@gonfunko
Labels
component: events component: touch issue: feature request Describes a new feature and why it should be added type: cleanup

Comments

@maribethb
Copy link
Contributor

Is your feature request related to a problem? Please describe.

Currently Blockly has complex code that tries to unify mouse and touch events. This is all custom code that predates the invention and subsequent adoption of PointerEvent by all major browsers. The custom code is complex and a source of bugs e.g. #6508

Describe the solution you'd like

Remove the custom code and switch to PointerEvents.

Describe alternatives you've considered

Additional context

Assigning to Aaron according to Q4 OKRs, please feel free to reassign if this changes.
@maribethb maribethb added type: cleanup component: touch issue: feature request Describes a new feature and why it should be added component: events issue: triage Issues awaiting triage by a Blockly team member labels Oct 13, 2022
@maribethb maribethb removed the issue: triage Issues awaiting triage by a Blockly team member label Oct 13, 2022
This was referenced Oct 13, 2022
Closed
Merged
@rachel-fenichel
Copy link
Collaborator

Internal doc with project description: https://docs.google.com/document/d/1vSlRNKOiS82jyqJr1VgEYcGwKEeUcqxPMS_YVoHZk60/edit?usp=drive_open&ouid=106800571011058844251

@gonfunko gonfunko mentioned this issue Dec 1, 2022
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gonfunko gonfunko

Labels
component: events component: touch issue: feature request Describes a new feature and why it should be added type: cleanup
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@gonfunko @maribethb @rachel-fenichel