-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Closed
flutter/engine
#43697Closed
Copy link
Labels
P1High-priority issues at the top of the work listHigh-priority issues at the top of the work listc: tech-debtTechnical debt, code quality, testing, etc.Technical debt, code quality, testing, etc.engineflutter/engine related. See also e: labels.flutter/engine related. See also e: labels.platform-webWeb applications specificallyWeb applications specificallyteam-webOwned by Web platform teamOwned by Web platform teamtriaged-webTriaged by Web platform teamTriaged by Web platform team
Description
The current web engine sources Pointer (mouse/tap) events from different APIs, depending on browser support:
- PointerEvent (caniuse -> preferred API)
- TouchEvent (caniuse -> fallback for Safari Mobile <13)
- MouseEvent (caniuse -> fallback for Safari Desktop <13)
The feature detection is also very trivial:
The latest release of Safari 12 (12.1.2) happened on:
- MacOS 10.14 (Mojave), which Apple hasn't supported since October 2021
- iOS 12.5.5, which Apple hasn't supported since August 2022
Proposal
Let's remove TouchEvents and MouseEvents as sources of PointerEvents from the pointer_binding.dart file above, to simplify it a lot, and maybe reduce a little bit the engine size along the way (this is minor).
Metadata
Metadata
Assignees
Labels
P1High-priority issues at the top of the work listHigh-priority issues at the top of the work listc: tech-debtTechnical debt, code quality, testing, etc.Technical debt, code quality, testing, etc.engineflutter/engine related. See also e: labels.flutter/engine related. See also e: labels.platform-webWeb applications specificallyWeb applications specificallyteam-webOwned by Web platform teamOwned by Web platform teamtriaged-webTriaged by Web platform teamTriaged by Web platform team