Skip to content

text selection for the web - #86664

Closed
jonahwilliams wants to merge 2 commits into
flutter:masterfrom
jonahwilliams:text_selection_1_of_n
Closed

jonahwilliams wants to merge 2 commits into
flutter:masterfrom
jonahwilliams:text_selection_1_of_n

Conversation

@jonahwilliams

@jonahwilliams jonahwilliams commented Jul 19, 2021 •

Copy link
Copy Markdown
Contributor

WIP due to testing / shortcuts questions.

Continuation of #81849

Work towards #81839

@flutter-dashboard flutter-dashboard Bot added the framework flutter/packages/flutter repository. See also f: labels. label Jul 19, 2021
@google-cla google-cla Bot added the cla: yes label Jul 19, 2021
ignoring: ignoreEvents,
child: child,
child: SelectionArea(
enabled: true, // Text selection is only enabled by default for the web.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Will set to kIsWeb, leaving it this way for testing

}

@override
Widget build(BuildContext context) {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@gspencergoog I feel like we talked about this before, but I'm unsure why I can't get these shortcuts to fire. i suppose I need the change that allows shortcut keys to fallback to parent shortcuts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant