Skip to content

Conversation

@gspencergoog
Copy link
Contributor

Description

This adds support to CharacterActivator for detecting the Alt key, needed for use in menu accelerator support.

It also adds a test for the ShortcutSerialization class, and adds support to that class's character constructor for all three modifiers (all except Shift).

Tests

  • Added tests for the Alt modifier, and tests for the ShortcutSerialization class.

@flutter-dashboard flutter-dashboard bot added the framework flutter/packages/flutter repository. See also f: labels. label Oct 14, 2022
@gspencergoog gspencergoog force-pushed the character_activator_alt branch 3 times, most recently from 3426960 to 4d8f2de Compare October 14, 2022 22:34
@gspencergoog gspencergoog force-pushed the character_activator_alt branch from 4d8f2de to 91661f5 Compare October 14, 2022 22:35
Copy link
Contributor

@dkwingsmt dkwingsmt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gspencergoog gspencergoog added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 17, 2022
@auto-submit auto-submit bot merged commit 1b4800c into flutter:master Oct 17, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Oct 18, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Oct 18, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Oct 18, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants