Fix/ios share context menu #176199

muradhossin · 2025-09-29T12:19:51Z

This PR fixes #173491 by adding the missing 'Share' option to the default iOS SystemContextMenu when shareEnabled is true.

Changes:

Added IOSSystemContextMenuItemShare to getDefaultItems in system_context_menu.dart.
Added a widget test to ensure Share is present in the default items for non-empty selections on iOS.
Rationale:
This aligns Flutter's default iOS text selection context menu with native iOS behavior, ensuring users see the expected 'Share' option when selecting text.

Demo:
Video showing Share option in iOS context menu: https://github.com/user-attachments/assets/e04cd1f9-7d92-4147-a09b-719f03d9c625

gemini-code-assist

Code Review

This pull request correctly adds the missing 'Share' option to the default iOS SystemContextMenu when shareEnabled is true, aligning Flutter's behavior with native iOS. The change is simple and effective, and it's accompanied by a thorough widget test to ensure the new functionality works as expected. I've included a few minor suggestions to improve code consistency and readability.

gemini-code-assist · 2025-09-29T12:21:43Z

packages/flutter/lib/src/widgets/system_context_menu.dart

      if (editableTextState.liveTextInputEnabled) const IOSSystemContextMenuItemLiveText(),
+      if (editableTextState.shareEnabled) const IOSSystemContextMenuItemShare(),


For consistency with the order of getters in EditableTextState, it would be better to place the shareEnabled check before liveTextInputEnabled. This improves logical grouping and maintainability, adhering to the style guide's principle of logical ordering.¹

if (editableTextState.shareEnabled) const IOSSystemContextMenuItemShare(), if (editableTextState.liveTextInputEnabled) const IOSSystemContextMenuItemLiveText(),

Style Guide References

Footnotes

Other class members should be ordered logically (e.g., by lifecycle, or grouping related fields and methods). (link) ↩

I would agree with that but no big deal if you have a reason for this ordering, @muradhossin .

gemini-code-assist · 2025-09-29T12:21:43Z

packages/flutter/test/widgets/system_context_menu_test.dart

+
+  testWidgets(
+    'Default iOS SystemContextMenu includes Share for non-empty selection',
+        (WidgetTester tester) async {


The indentation for the test callback seems off. Running dart format should fix this and ensure consistency with the rest of the file. This adheres to the Flutter style guide which mandates dart format usage.¹

Suggested change

(WidgetTester tester) async {

(WidgetTester tester) async {

Style Guide References

Footnotes

All Dart code is formatted using dart format. This is enforced by CI. (link) ↩

Not sure if this is right or not, I'll defer to the CI checks. If they're green it's ok.

gemini-code-assist · 2025-09-29T12:21:44Z

packages/flutter/test/widgets/system_context_menu_test.dart

+            final List<IOSSystemContextMenuItemData> lastItems = untypedItems.map((dynamic value) {
+              final Map<String, dynamic> itemJson = value as Map<String, dynamic>;
+              return systemContextMenuItemDataFromJson(itemJson);
+            }).toList();


This mapping can be simplified for better readability by using a collection for.

Suggested change

final List<IOSSystemContextMenuItemData> lastItems = untypedItems.map((dynamic value) {

final Map<String, dynamic> itemJson = value as Map<String, dynamic>;

return systemContextMenuItemDataFromJson(itemJson);

}).toList();

final List<IOSSystemContextMenuItemData> lastItems = <IOSSystemContextMenuItemData>[

for (final dynamic value in untypedItems)

systemContextMenuItemDataFromJson(value as Map<String, dynamic>),

];

This seems legit.

justinmc

Thanks for fixing this. Did we just forget to add the share button here? It makes me really wish we could use a switch or something to ensure that we don't miss any other new subclasses.

Take a look at the analyzer failure, you're missing a type in your test.

justinmc · 2025-10-21T23:18:46Z

packages/flutter/lib/src/widgets/system_context_menu.dart

      if (editableTextState.liveTextInputEnabled) const IOSSystemContextMenuItemLiveText(),
+      if (editableTextState.shareEnabled) const IOSSystemContextMenuItemShare(),


I would agree with that but no big deal if you have a reason for this ordering, @muradhossin .

justinmc · 2025-10-21T23:21:17Z

packages/flutter/test/widgets/system_context_menu_test.dart

+
+  testWidgets(
+    'Default iOS SystemContextMenu includes Share for non-empty selection',
+        (WidgetTester tester) async {


Not sure if this is right or not, I'll defer to the CI checks. If they're green it's ok.

justinmc · 2025-10-21T23:21:49Z

packages/flutter/test/widgets/system_context_menu_test.dart

+            final List<IOSSystemContextMenuItemData> lastItems = untypedItems.map((dynamic value) {
+              final Map<String, dynamic> itemJson = value as Map<String, dynamic>;
+              return systemContextMenuItemDataFromJson(itemJson);
+            }).toList();


This seems legit.

justinmc

Looks good besides the accidental changes in pubspec.lock.

justinmc · 2025-10-28T21:48:37Z

packages/flutter/lib/src/widgets/system_context_menu.dart

+    for (final ContextMenuButtonItem button in editableTextState.contextMenuButtonItems) {
+      switch (button.type) {


This is great, now we know all button types are handled and always will be!

justinmc · 2025-10-28T21:50:56Z

packages/flutter/test/widgets/system_context_menu_test.dart

+      expect(
+        itemsReceived.last.any(
+          (IOSSystemContextMenuItemData e) => e is IOSSystemContextMenuItemDataShare,
+        ),
+        isTrue,
+      );


Nit: I wonder if there is a matcher that you could use here like contains? See https://main-api.flutter.dev/flutter/package-matcher_matcher/.

justinmc · 2025-10-28T21:51:22Z

pubspec.lock

Can you revert the changes in this file?

justinmc

LGTM 👍, thanks for following through on this PR!

jmagman · 2025-11-06T00:49:21Z

packages/flutter/lib/src/widgets/system_context_menu.dart

+    for (final ContextMenuButtonItem button in editableTextState.contextMenuButtonItems) {
+      switch (button.type) {
+        case ContextMenuButtonType.copy:
+          items.add(const IOSSystemContextMenuItemCopy());


Does editableTextState.copyEnabled etc not need to be checked any more?

contextMenuButtonItems already applies those enable-checks for us.

So by iterating editableTextState.contextMenuButtonItems, we’re consuming the filtered set — only enabled actions are present.

That’s why we no longer need separate editableTextState.copyEnabled etc checks here — the switch is now operating on the already-screened model, which is the single source of truth for both Flutter-rendered and native menus.

muradhossin · 2025-11-19T05:28:01Z

@justinmc May I know if there’s any update on the merge?

dkwingsmt

LGTM. Thank you!

justinmc · 2025-12-30T23:36:30Z

@muradhossin Can you fix the analyzer failure here?

Fixes flutter#173491. This adds IOSSystemContextMenuItemShare to the default iOS system context menu items when shareEnabled is true, ensuring native 'Share' option appears for text selections.

# Conflicts: # pubspec.lock

muradhossin · 2025-12-31T13:21:47Z

@muradhossin Can you fix the analyzer failure here?

@justinmc Could you please guide me on why many of the checks are failing?

LouiseHsu · 2026-01-05T20:37:20Z

@muradhossin Can you fix the analyzer failure here?

@justinmc Could you please guide me on why many of the checks are failing?

can you try removing your changes to pubspec.lock and then merge master?

Roll Flutter from 13b2b91 to 01d37bc (74 revisions) flutter/flutter@13b2b91...01d37bc 2026-01-07 stuartmorgan@google.com Replace Hybrid Composition wiki page with dev-facing content (flutter/flutter#180642) 2026-01-07 116356835+AbdeMohlbi@users.noreply.github.com Improve code quality in `FlutterActivityTest.java` (flutter/flutter#180585) 2026-01-07 iinozemtsev@google.com Roll Dart SDK to 3.11.0-296.1.beta (flutter/flutter#180633) 2026-01-07 vegorov@google.com Bump target Windows version to 10 (flutter/flutter#180624) 2026-01-07 goderbauer@google.com Run hook_user_defines and link_hook integration tests on CI (flutter/flutter#180622) 2026-01-07 iliyazelenkog@gmail.com Fix division by zero in RenderTable intrinsic size methods (flutter/flutter#178217) 2026-01-07 116356835+AbdeMohlbi@users.noreply.github.com Remove more `requires 24` anotations (flutter/flutter#180116) 2026-01-07 engine-flutter-autoroll@skia.org Roll Skia from 3971dbb85d45 to c5359a4d720e (1 revision) (flutter/flutter#180631) 2026-01-07 huy@nevercode.io Fix TabBar.image does not render at initialIndex for the first time (flutter/flutter#179616) 2026-01-07 engine-flutter-autoroll@skia.org Roll Skia from 1e7ad625f6f7 to 3971dbb85d45 (3 revisions) (flutter/flutter#180627) 2026-01-07 goderbauer@google.com Unpin DDS (flutter/flutter#180571) 2026-01-07 bruno.leroux@gmail.com Fix DropdownMenuEntry.style not resolved when entry is highlighted (flutter/flutter#178873) 2026-01-07 116356835+AbdeMohlbi@users.noreply.github.com Remove unnecessary `@RequiresApi24` annotations from FlutterFragment methods (flutter/flutter#180117) 2026-01-07 engine-flutter-autoroll@skia.org Roll Skia from 7fc63228056b to 1e7ad625f6f7 (1 revision) (flutter/flutter#180616) 2026-01-07 engine-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from QfR2ZFZ5kGTD3raO3... to dTvN_JVSCfGFRasvH... (flutter/flutter#180612) 2026-01-07 bkonyi@google.com [ Widget Preview ] Add support for `dart:ffi` imports (flutter/flutter#180586) 2026-01-07 engine-flutter-autoroll@skia.org Roll Skia from eec90000a899 to 7fc63228056b (2 revisions) (flutter/flutter#180608) 2026-01-07 ulisses.hen@hotmail.com Add --web-define flag for template variable injection in Flutter web builds (flutter/flutter#175805) 2026-01-07 codefu@google.com docs(engine): update rbe notes (flutter/flutter#180599) 2026-01-06 engine-flutter-autoroll@skia.org Roll Skia from b6249496c230 to eec90000a899 (3 revisions) (flutter/flutter#180602) 2026-01-06 146823759+brahim-guaali@users.noreply.github.com Forward proxy 404 responses to client (flutter/flutter#179858) 2026-01-06 engine-flutter-autoroll@skia.org Roll Dart SDK from 40a8c6188f7f to d2e3ce177270 (1 revision) (flutter/flutter#180598) 2026-01-06 104009581+Saqib198@users.noreply.github.com Restore CLI precedence for web headers and HTTPS over web_dev_config.yaml (flutter/flutter#179639) 2026-01-06 engine-flutter-autoroll@skia.org Roll Skia from f5d9da13d56d to b6249496c230 (1 revision) (flutter/flutter#180596) 2026-01-06 dkwingsmt@users.noreply.github.com Enable misc leak tracking (flutter/flutter#176992) 2026-01-06 dacoharkes@google.com [hooks] Don't require NDK for Android targets (flutter/flutter#180594) 2026-01-06 kjlubick@users.noreply.github.com [skia] Disable legacy non-const SkData APIs (flutter/flutter#179684) 2026-01-06 48625061+muradhossin@users.noreply.github.com Fix/ios share context menu (flutter/flutter#176199) 2026-01-06 engine-flutter-autoroll@skia.org Roll Skia from b970aeffa66f to f5d9da13d56d (5 revisions) (flutter/flutter#180588) 2026-01-06 goderbauer@google.com Manually bump dependencies (flutter/flutter#180509) 2026-01-06 engine-flutter-autoroll@skia.org Roll Dart SDK from 8150be8a0e48 to 40a8c6188f7f (2 revisions) (flutter/flutter#180582) 2026-01-06 engine-flutter-autoroll@skia.org Roll Packages from 12eb764 to d3f860d (2 revisions) (flutter/flutter#180581) 2026-01-06 engine-flutter-autoroll@skia.org Roll Fuchsia Test Scripts from MHF-UAfO6sVKqSEYk... to nR2ESa1Gd8yPcWo06... (flutter/flutter#180578) 2026-01-06 sokolovskyi.konstantin@gmail.com Add tooltip support to PlatformMenuItem and PlatformMenu. (flutter/flutter#180069) 2026-01-06 bruno.leroux@gmail.com Add DropdownMenuFormField.errorBuilder (flutter/flutter#179345) 2026-01-06 engine-flutter-autoroll@skia.org Roll Skia from 1845397e11ba to b970aeffa66f (2 revisions) (flutter/flutter#180566) 2026-01-06 oss@simonbinder.eu Don't embed unreferenced assets (flutter/flutter#179251) 2026-01-06 116356835+AbdeMohlbi@users.noreply.github.com Improve documentation about ValueNotifier's behavior (flutter/flutter#179870) 2026-01-06 engine-flutter-autoroll@skia.org Roll Skia from 904ba00331ca to 1845397e11ba (5 revisions) (flutter/flutter#180558) 2026-01-06 engine-flutter-autoroll@skia.org Roll Dart SDK from 2fb9ad834c4d to 8150be8a0e48 (1 revision) (flutter/flutter#180557) 2026-01-06 engine-flutter-autoroll@skia.org Roll Skia from 98c042dde68c to 904ba00331ca (3 revisions) (flutter/flutter#180550) 2026-01-06 engine-flutter-autoroll@skia.org Roll Dart SDK from ba9f7f790966 to 2fb9ad834c4d (2 revisions) (flutter/flutter#180548) 2026-01-06 engine-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from ubBGcRaAKWKihQ4ac... to QfR2ZFZ5kGTD3raO3... (flutter/flutter#180547) 2026-01-06 ahmedsameha1@gmail.com Make sure that a DraggableScrollableSheet doesn't crash in 0x0 enviro… (flutter/flutter#180433) 2026-01-06 ahmedsameha1@gmail.com Make sure that a ColorFiltered doesn't crash 0x0 environment (flutter/flutter#180307) 2026-01-06 ahmedsameha1@gmail.com Make sure that a FadeInImage doesn't crash in 0x0 environment (flutter/flutter#180495) ...

github-actions bot added the framework flutter/packages/flutter repository. See also f: labels. label Sep 29, 2025

gemini-code-assist bot reviewed Sep 29, 2025

View reviewed changes

justinmc self-requested a review October 14, 2025 22:19

justinmc reviewed Oct 21, 2025

View reviewed changes

justinmc reviewed Oct 28, 2025

View reviewed changes

justinmc approved these changes Nov 5, 2025

View reviewed changes

jmagman reviewed Nov 6, 2025

View reviewed changes

justinmc requested a review from dkwingsmt November 25, 2025 23:09

dkwingsmt approved these changes Dec 15, 2025

View reviewed changes

muradhossin added 4 commits December 31, 2025 14:57

feat(cupertino): include Share in iOS SystemContextMenu defaults

95db54d

Fixes flutter#173491. This adds IOSSystemContextMenuItemShare to the default iOS system context menu items when shareEnabled is true, ensuring native 'Share' option appears for text selections.

fix(widgets): address review feedback for iOS SystemContextMenu

f798ce3

# Conflicts: # pubspec.lock

chore: revert pubspec.lock to upstream main

9fbfa8d

fix(tests): use matcher for Share presence in SystemContextMenu defaults

b980787

muradhossin requested review from a team and jtmcdole as code owners December 31, 2025 09:06

Merge branch 'master' into fix/ios-share-context-menu

9affd6e

muradhossin and others added 2 commits January 6, 2026 14:59

Merge branch 'master' into fix/ios-share-context-menu

0a82dbb

Update pubspec.lock from upstream master

db44eb8

LouiseHsu added the autosubmit Merge PR when tree becomes green via auto submit App label Jan 6, 2026

auto-submit bot added this pull request to the merge queue Jan 6, 2026

Merged via the queue into flutter:master with commit eb7fb9a Jan 6, 2026
71 checks passed

flutter-dashboard bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Jan 6, 2026

engine-flutter-autoroll mentioned this pull request Jan 7, 2026

Roll Flutter from 13b2b912c909 to 20fc4107d61f (67 revisions) flutter/packages#10740

Closed

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

eb7fb9a Fix/ios share context menu (flutter/flutter#176199)

a9e3b15

engine-flutter-autoroll mentioned this pull request Jan 7, 2026

Roll Flutter from 13b2b912c909 to a73c1002a6fa (68 revisions) flutter/packages#10741

Closed

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

eb7fb9a Fix/ios share context menu (flutter/flutter#176199)

bc9c3d6

engine-flutter-autoroll mentioned this pull request Jan 7, 2026

Roll Flutter from 13b2b912c909 to b2d157623eeb (69 revisions) flutter/packages#10742

Closed

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

eb7fb9a Fix/ios share context menu (flutter/flutter#176199)

7ba3cb9

engine-flutter-autoroll mentioned this pull request Jan 7, 2026

Roll Flutter from 13b2b912c909 to 6acc84ee2ce9 (71 revisions) flutter/packages#10743

Closed

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

eb7fb9a Fix/ios share context menu (flutter/flutter#176199)

b39790b

engine-flutter-autoroll mentioned this pull request Jan 7, 2026

Roll Flutter from 13b2b912c909 to 01d37bc80fd5 (74 revisions) flutter/packages#10745

Merged

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

eb7fb9a Fix/ios share context menu (flutter/flutter#176199)

dab363b

		if (editableTextState.liveTextInputEnabled) const IOSSystemContextMenuItemLiveText(),
		if (editableTextState.shareEnabled) const IOSSystemContextMenuItemShare(),

		for (final ContextMenuButtonItem button in editableTextState.contextMenuButtonItems) {
		switch (button.type) {

Fix/ios share context menu #176199

Fix/ios share context menu #176199

Conversation

muradhossin commented Sep 29, 2025

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist bot Sep 29, 2025

Choose a reason for hiding this comment

Style Guide References

Footnotes

Uh oh!

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Sep 29, 2025

Choose a reason for hiding this comment

Style Guide References

Footnotes

Uh oh!

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Sep 29, 2025

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

justinmc left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

justinmc left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

justinmc left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

muradhossin commented Nov 19, 2025

Uh oh!

dkwingsmt left a comment

Choose a reason for hiding this comment

Uh oh!

justinmc commented Dec 30, 2025

Uh oh!

muradhossin commented Dec 31, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

LouiseHsu commented Jan 5, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

muradhossin commented Dec 31, 2025 •

edited

Loading