-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Closed
Labels
P1High-priority issues at the top of the work listHigh-priority issues at the top of the work lista: desktopRunning on desktopRunning on desktopa: text inputEntering text in a text field or keyboard related problemsEntering text in a text field or keyboard related problemsengineflutter/engine related. See also e: labels.flutter/engine related. See also e: labels.frameworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.team-text-inputOwned by Text Input teamOwned by Text Input team
Milestone
Description
In order to make implementing #33521 easier, and because it is a better factoring of the code, it makes sense to have the embedding side translate the local key events into a generic representation that the framework can then pass along.
This could be done by moving all of the keyboard key translation to the engine side, normalize it there, and just send a generic key event to the framework.
This would be a large breaking change on the framework side to remove the ability to look at the raw key event data on the framework side, but that is a fairly uncommon operation. We would have to make sure that we encoded all the relevant information in the generic key event (key repeat counts, left/right modifier information, etc.).
Leffe108, Wishmater, ReniDelonzek, escamoteur, rydmike and 4 more
Metadata
Metadata
Assignees
Labels
P1High-priority issues at the top of the work listHigh-priority issues at the top of the work lista: desktopRunning on desktopRunning on desktopa: text inputEntering text in a text field or keyboard related problemsEntering text in a text field or keyboard related problemsengineflutter/engine related. See also e: labels.flutter/engine related. See also e: labels.frameworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.team-text-inputOwned by Text Input teamOwned by Text Input team