`SelectableRegion` should not show flutter rendered context menu when web context menu is enabled by Renzo-Olivares · Pull Request #176855 · flutter/flutter

Renzo-Olivares · 2025-10-10T19:01:58Z

This change fixes an issue where the flutter rendered menu would show up when ending a drag on a selection handle, even when the web context menu was enabled. This led to issues where both context menus would show up leading to a unfavorable experience.

Fixes #175114

Pre-launch Checklist

I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
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.

gemini-code-assist

Code Review

This pull request correctly fixes an issue where the Flutter-rendered context menu would appear on top of the native web context menu after a selection handle drag. The approach of centralizing the toolbar showing logic in _showToolbar and using a _webContextMenuEnabled getter is clean.

I've left a couple of comments: one suggestion to simplify the code by removing a now-redundant conditional block, and another pointing out a potential bug in the new handlesAreVisible getter which could cause issues with handle visibility checks.

chunhtai · 2025-10-13T17:52:48Z

        _selectionOverlay != null &&
        (_selectionOverlay!.isDraggingStartHandle || _selectionOverlay!.isDraggingEndHandle);
-    if (widget.selectionControls is! TextSelectionHandleControls) {
-      if (!draggingHandles) {


Why this change, I think this may be code to handle custom context menu (although we shouldn't probably use runtimetype check to infer this)

It ends up being the same before/after. The code to handle the context menu TextSelectionControls vs ContextMenuBuilder is in _showToolbar.

oh I see, sounds good then. Can you add a todo in the runtitmeType check in _showToolbar to refactor that?

loic-sharma

LGTM!

chunhtai

LGTM

chunhtai · 2025-10-13T19:45:24Z

        _selectionOverlay != null &&
        (_selectionOverlay!.isDraggingStartHandle || _selectionOverlay!.isDraggingEndHandle);
-    if (widget.selectionControls is! TextSelectionHandleControls) {
-      if (!draggingHandles) {


oh I see, sounds good then. Can you add a todo in the runtitmeType check in _showToolbar to refactor that?

justinmc

LGTM 👍

justinmc · 2025-10-13T22:55:55Z

+  /// Whether the native browser context menu is enabled.
+  bool get _webContextMenuEnabled => kIsWeb && BrowserContextMenu.enabled;


Ah same strategy used in EditableText. More evidence that having two separate implementations of text selection leads to bugs.

auto-submit · 2025-10-14T18:01:46Z

autosubmit label was removed for flutter/flutter/176855, because - The status or check suite Mac_arm64 build_tests_3_4 has failed. Please fix the issues identified (or deflake) before re-applying this label.

The status or check suite Mac_arm64 build_tests_2_4 has failed. Please fix the issues identified (or deflake) before re-applying this label.

…electionOverlay to prevent double context menus

… is fully removed

…enu when web context menu is enabled (flutter/flutter#176855)

Manual roll requested by tarrinneal@google.com flutter/flutter@7cd821c...a873a27 2025-10-16 30870216+gaaclarke@users.noreply.github.com [tool] makes listing a shader also as an asset a build failure (flutter/flutter#176866) 2025-10-16 engine-flutter-autoroll@skia.org Roll Packages from d062181 to 835dccb (7 revisions) (flutter/flutter#177100) 2025-10-16 jason-simmons@users.noreply.github.com Handle the new location of Perfetto in create_updated_flutter_deps.py (flutter/flutter#177099) 2025-10-16 matt.kosarek@canonical.com Implement dialog windows for the win32 platform (flutter/flutter#176309) 2025-10-16 rmolivares@renzo-olivares.dev `SelectableRegion` should not show flutter rendered context menu when web context menu is enabled (flutter/flutter#176855) 2025-10-16 jason-simmons@users.noreply.github.com Manual roll Skia to 2d9df7c70b6f (flutter/flutter#177074) 2025-10-16 31685655+SalehTZ@users.noreply.github.com feat: add `OptionsViewOpenDirection.mostSpace` to `RawAutocomplete` (flutter/flutter#172997) 2025-10-16 43054281+camsim99@users.noreply.github.com [Android] Refactor `ImageReaderSurfaceProducer` restoration after app resumes (flutter/flutter#175937) 2025-10-15 34465683+rkishan516@users.noreply.github.com Refactor: migrate fade upwards page transition builder to widgets (flutter/flutter#175560) 2025-10-15 fluttergithubbot@gmail.com Marks Windows windowing_test to be unflaky (flutter/flutter#176701) 2025-10-15 72062416+Yash-Dhrangdhariya@users.noreply.github.com fix: 🐛 Add equality and hashCode implementations to ScrollAwareImageProvider (flutter/flutter#175038) 2025-10-15 mohellebiabdessalem@gmail.com Updates `sliver_tree.1.dart‎` to use `MediaQuery.widthOf(context)` (flutter/flutter#176888) 2025-10-15 mdebbar@google.com [web] Fix focus issues in newer versions of Chrome (flutter/flutter#176938) 2025-10-15 jessiewong401@gmail.com [Android 16] Update `android_engine_vulkan_tests` to Test Against SDK 36 Emulator (flutter/flutter#176985) 2025-10-15 sokolovskyi.konstantin@gmail.com Fix key events interception by RadioGroup when no Radio is focused. (flutter/flutter#176335) 2025-10-15 21270878+elliette@users.noreply.github.com Update cherry-pick instructions to include instructions for pre-release CPs (flutter/flutter#177020) 2025-10-15 jason-simmons@users.noreply.github.com Manual roll Skia to c501c727a007 (flutter/flutter#177015) 2025-10-15 robert.ancell@canonical.com Update examples to latest Linux runner style (flutter/flutter#177033) 2025-10-15 59215665+davidhicks980@users.noreply.github.com [material/menu_anchor.dart] Create internal menu controller if external controller is changed to null. (flutter/flutter#176375) 2025-10-15 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Fix - TalkBack does not announce list information (#174374)" (flutter/flutter#177062) 2025-10-14 robert.ancell@canonical.com Implement Regular Windows for Linux (flutter/flutter#176187) 2025-10-14 31510811+jwlilly@users.noreply.github.com Fix - TalkBack does not announce list information (flutter/flutter#174374) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC stuartmorgan@google.com,tarrinneal@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

… web context menu is enabled (flutter#176855) This change fixes an issue where the flutter rendered menu would show up when ending a drag on a selection handle, even when the web context menu was enabled. This led to issues where both context menus would show up leading to a unfavorable experience. Fixes flutter#175114 ## Pre-launch Checklist - [x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [x] I read the [Tree Hygiene] wiki page, which explains my responsibilities. - [x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement]. - [x] I signed the [CLA]. - [x] I listed at least one issue that this PR fixes in the description above. - [x] I updated/added relevant documentation (doc comments with `///`). - [x] I added new tests to check the change I am making, or this PR is [test-exempt]. - [x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported. - [x] All existing and new tests are passing. --------- Co-authored-by: Renzo Olivares <roliv@google.com>

…enu when web context menu is enabled (flutter/flutter#176855)

…0244) Manual roll requested by tarrinneal@google.com flutter/flutter@7cd821c...a873a27 2025-10-16 30870216+gaaclarke@users.noreply.github.com [tool] makes listing a shader also as an asset a build failure (flutter/flutter#176866) 2025-10-16 engine-flutter-autoroll@skia.org Roll Packages from d062181 to 835dccb (7 revisions) (flutter/flutter#177100) 2025-10-16 jason-simmons@users.noreply.github.com Handle the new location of Perfetto in create_updated_flutter_deps.py (flutter/flutter#177099) 2025-10-16 matt.kosarek@canonical.com Implement dialog windows for the win32 platform (flutter/flutter#176309) 2025-10-16 rmolivares@renzo-olivares.dev `SelectableRegion` should not show flutter rendered context menu when web context menu is enabled (flutter/flutter#176855) 2025-10-16 jason-simmons@users.noreply.github.com Manual roll Skia to 2d9df7c70b6f (flutter/flutter#177074) 2025-10-16 31685655+SalehTZ@users.noreply.github.com feat: add `OptionsViewOpenDirection.mostSpace` to `RawAutocomplete` (flutter/flutter#172997) 2025-10-16 43054281+camsim99@users.noreply.github.com [Android] Refactor `ImageReaderSurfaceProducer` restoration after app resumes (flutter/flutter#175937) 2025-10-15 34465683+rkishan516@users.noreply.github.com Refactor: migrate fade upwards page transition builder to widgets (flutter/flutter#175560) 2025-10-15 fluttergithubbot@gmail.com Marks Windows windowing_test to be unflaky (flutter/flutter#176701) 2025-10-15 72062416+Yash-Dhrangdhariya@users.noreply.github.com fix: 🐛 Add equality and hashCode implementations to ScrollAwareImageProvider (flutter/flutter#175038) 2025-10-15 mohellebiabdessalem@gmail.com Updates `sliver_tree.1.dart‎` to use `MediaQuery.widthOf(context)` (flutter/flutter#176888) 2025-10-15 mdebbar@google.com [web] Fix focus issues in newer versions of Chrome (flutter/flutter#176938) 2025-10-15 jessiewong401@gmail.com [Android 16] Update `android_engine_vulkan_tests` to Test Against SDK 36 Emulator (flutter/flutter#176985) 2025-10-15 sokolovskyi.konstantin@gmail.com Fix key events interception by RadioGroup when no Radio is focused. (flutter/flutter#176335) 2025-10-15 21270878+elliette@users.noreply.github.com Update cherry-pick instructions to include instructions for pre-release CPs (flutter/flutter#177020) 2025-10-15 jason-simmons@users.noreply.github.com Manual roll Skia to c501c727a007 (flutter/flutter#177015) 2025-10-15 robert.ancell@canonical.com Update examples to latest Linux runner style (flutter/flutter#177033) 2025-10-15 59215665+davidhicks980@users.noreply.github.com [material/menu_anchor.dart] Create internal menu controller if external controller is changed to null. (flutter/flutter#176375) 2025-10-15 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Fix - TalkBack does not announce list information (#174374)" (flutter/flutter#177062) 2025-10-14 robert.ancell@canonical.com Implement Regular Windows for Linux (flutter/flutter#176187) 2025-10-14 31510811+jwlilly@users.noreply.github.com Fix - TalkBack does not announce list information (flutter/flutter#174374) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC stuartmorgan@google.com,tarrinneal@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

github-actions Bot added a: text input Entering text in a text field or keyboard related problems framework flutter/packages/flutter repository. See also f: labels. labels Oct 10, 2025

gemini-code-assist Bot reviewed Oct 10, 2025

View reviewed changes

Comment thread packages/flutter/lib/src/widgets/text_selection.dart Outdated

Comment thread packages/flutter/lib/src/widgets/selectable_region.dart Outdated

Renzo-Olivares changed the title ~~SelectableRegion should not show flutter rendered context menu when web context menu is enabled~~ SelectableRegion should not show flutter rendered context menu when web context menu is enabled Oct 10, 2025

github-actions Bot removed the a: text input Entering text in a text field or keyboard related problems label Oct 12, 2025

Renzo-Olivares requested review from chunhtai, justinmc and loic-sharma October 13, 2025 16:22

chunhtai reviewed Oct 13, 2025

View reviewed changes

loic-sharma approved these changes Oct 13, 2025

View reviewed changes

chunhtai approved these changes Oct 13, 2025

View reviewed changes

justinmc approved these changes Oct 13, 2025

View reviewed changes

Renzo-Olivares added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 14, 2025

auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Oct 14, 2025

Renzo Olivares added 6 commits October 15, 2025 21:33

Call _showToolbar method instead of directly calling showToolbar on S…

1200a33

…electionOverlay to prevent double context menus

updates and add test

7ae5062

updates

233c00f

fix

9f40621

updates

ca6613a

add TODO to remove runtimeType check when TextSelectionHandleControls…

941376a

… is fully removed

Renzo-Olivares added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 16, 2025

auto-submit Bot added this pull request to the merge queue Oct 16, 2025

Merged via the queue into flutter:master with commit 666634b Oct 16, 2025
78 checks passed

flutter-dashboard Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Oct 16, 2025

engine-flutter-autoroll mentioned this pull request Oct 16, 2025

Roll Flutter from 7cd821c21093 to 666634bffb6b (18 revisions) flutter/packages#10240

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 16, 2025

666634b SelectableRegion should not show flutter rendered context m…

c7bcff8

…enu when web context menu is enabled (flutter/flutter#176855)

engine-flutter-autoroll mentioned this pull request Oct 16, 2025

Roll Flutter from 7cd821c21093 to fe409cfca41f (19 revisions) flutter/packages#10241

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 16, 2025

666634b SelectableRegion should not show flutter rendered context m…

6b0a451

…enu when web context menu is enabled (flutter/flutter#176855)

engine-flutter-autoroll mentioned this pull request Oct 16, 2025

Roll Flutter from 7cd821c21093 to a27b7cda85fb (21 revisions) flutter/packages#10242

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 16, 2025

666634b SelectableRegion should not show flutter rendered context m…

9590122

…enu when web context menu is enabled (flutter/flutter#176855)

engine-flutter-autoroll mentioned this pull request Oct 16, 2025

Roll Flutter from 7cd821c21093 to a873a27309b0 (22 revisions) flutter/packages#10243

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 16, 2025

666634b SelectableRegion should not show flutter rendered context m…

223b91e

…enu when web context menu is enabled (flutter/flutter#176855)

engine-flutter-autoroll mentioned this pull request Oct 16, 2025

Manual roll Flutter from 7cd821c21093 to a873a27309b0 (22 revisions) flutter/packages#10244

Merged

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 16, 2025

666634b SelectableRegion should not show flutter rendered context m…

8651982

…enu when web context menu is enabled (flutter/flutter#176855)

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 11, 2026

666634b SelectableRegion should not show flutter rendered context m…

67f9ab7

…enu when web context menu is enabled (flutter/flutter#176855)

xsahil03x mentioned this pull request Aug 20, 2026

fix(ui): keep web text selection stable across browser context menu toggles GetStream/stream-chat-flutter#2909

Merged

3 tasks done

Navigation Menu

Repository navigation

`SelectableRegion` should not show flutter rendered context menu when web context menu is enabled - #176855

`SelectableRegion` should not show flutter rendered context menu when web context menu is enabled#176855
auto-submit[bot] merged 6 commits into
flutter:masterfrom
Renzo-Olivares:sr-double-context-menu-fix

Renzo-Olivares commented Oct 10, 2025 •

edited

Loading

Uh oh!

gemini-code-assist Bot left a comment

Uh oh!

Uh oh!

Uh oh!

chunhtai Oct 13, 2025

Uh oh!

Renzo-Olivares Oct 13, 2025

Uh oh!

chunhtai Oct 13, 2025 •

edited

Loading

Uh oh!

loic-sharma left a comment

Uh oh!

chunhtai left a comment

Uh oh!

chunhtai Oct 13, 2025 •

edited

Loading

Uh oh!

justinmc left a comment

Uh oh!

justinmc Oct 13, 2025

Uh oh!

auto-submit Bot commented Oct 14, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

		/// Whether the native browser context menu is enabled.
		bool get _webContextMenuEnabled => kIsWeb && BrowserContextMenu.enabled;

Conversation

Renzo-Olivares commented Oct 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pre-launch Checklist

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

Uh oh!

chunhtai Oct 13, 2025

Choose a reason for hiding this comment

Uh oh!

Renzo-Olivares Oct 13, 2025

Choose a reason for hiding this comment

Uh oh!

chunhtai Oct 13, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

loic-sharma left a comment

Choose a reason for hiding this comment

Uh oh!

chunhtai left a comment

Choose a reason for hiding this comment

Uh oh!

chunhtai Oct 13, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

justinmc left a comment

Choose a reason for hiding this comment

Uh oh!

justinmc Oct 13, 2025

Choose a reason for hiding this comment

Uh oh!

auto-submit Bot commented Oct 14, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Renzo-Olivares commented Oct 10, 2025 •

edited

Loading

chunhtai Oct 13, 2025 •

edited

Loading

chunhtai Oct 13, 2025 •

edited

Loading