Skip to content

SelectableRegion can dismiss context menu with keyboard shortcuts#184788

Merged
auto-submit[bot] merged 3 commits intoflutter:masterfrom
Renzo-Olivares:dismiss-intent-selectableregion
Apr 11, 2026
Merged

SelectableRegion can dismiss context menu with keyboard shortcuts#184788
auto-submit[bot] merged 3 commits intoflutter:masterfrom
Renzo-Olivares:dismiss-intent-selectableregion

Conversation

@Renzo-Olivares
Copy link
Copy Markdown
Contributor

@Renzo-Olivares Renzo-Olivares commented Apr 8, 2026

Part of #182628

Adds support for DismissIntent in SelectableRegion, so it may dismiss the context menu when it is present and the "esc" key is pressed.

Pre-launch Checklist

  • I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • I read the [AI contribution guidelines] and understand my responsibilities, or I am not using AI tools.
  • I read the [Tree Hygiene] wiki page, which explains my responsibilities.
  • I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
  • I signed the [CLA].
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is [test-exempt].
  • I followed the [breaking change policy] and added [Data Driven Fixes] where supported.
  • All existing and new tests are passing.

@github-actions github-actions bot added the framework flutter/packages/flutter repository. See also f: labels. label Apr 8, 2026
@Renzo-Olivares Renzo-Olivares marked this pull request as ready for review April 8, 2026 21:44
@Renzo-Olivares Renzo-Olivares added the CICD Run CI/CD label Apr 8, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a DismissIntent handler to SelectableRegion to allow hiding the toolbar via keyboard shortcuts, along with a corresponding test case. The review identifies a potential crash when invoking the intent and suggests using Actions.maybeInvoke instead, while also recommending the addition of documentation for the new private method to align with repository style guidelines.

Copy link
Copy Markdown
Member

@loic-sharma loic-sharma left a comment

Choose a reason for hiding this comment

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

Nice fix!

Copy link
Copy Markdown
Contributor

@chunhtai chunhtai left a comment

Choose a reason for hiding this comment

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

LGTM

@Renzo-Olivares Renzo-Olivares added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 11, 2026
@auto-submit auto-submit bot added this pull request to the merge queue Apr 11, 2026
Merged via the queue into flutter:master with commit bf9f14b Apr 11, 2026
85 checks passed
@flutter-dashboard flutter-dashboard bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Apr 11, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 11, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 11, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 11, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 12, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 12, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants