Cupertino ContextMenu improved onTap support

Use case

Many of the native iOS apps that support the CupertinoContextMenu use it on items that already have an existing onTap implementation.

Proposal

Currently, CupertinoContextMenu can trigger while also triggering onTap of the child element (depending on the exact duration of the tap) so it'd be nice to rework this such that a single GestureDetector is controlling both.

same issue #70716

@cpboyd

@cpboyd Did you come up with any workaround? I calling a package with a native iOS implementation, but it's very complicated.

It looks like this was resolved by #131030. If this issue is still present, please file a new one with updated reproduction steps. Thank you!

This issue is still present, it is really evident when pushing a route because the overlay entry stays on top of the new route. To reproduce you have to hold the tap for a bit too long, but not long enough to trigger the context menu, see the third "tap" in the video.

Screen.Recording.2025-05-26.at.21.28.48.mov

Here is a sample code to reproduce this issue:

import 'package:flutter/cupertino.dart';
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    final colorScheme = ColorScheme.fromSeed(seedColor: Colors.orange);
    return MaterialApp(
      theme: ThemeData(
        colorScheme: colorScheme,
        appBarTheme: AppBarTheme(backgroundColor: colorScheme.secondaryContainer),
      ),
      home: const HomePage(),
    );
  }
}

class HomePage extends StatelessWidget {
  const HomePage({super.key});

  @override
  Widget build(BuildContext context) => Scaffold(
        appBar: AppBar(
          title: Text('Home'),
        ),
        body: Center(
          child: CupertinoContextMenu(
            actions: [
              CupertinoContextMenuAction(
                child: Text('Test'),
              ),
            ],
            child: GestureDetector(
              onTap: () {
                Navigator.of(context).push(
                  MaterialPageRoute(builder: (context) => _OtherPage()),
                );
              },
              child: Container(
                color: Colors.orange,
                height: 100,
                width: 100,
              ),
            ),
          ),
        ),
      );
}

class _OtherPage extends StatelessWidget {
  const _OtherPage({super.key});

  @override
  Widget build(BuildContext context) => Scaffold(
        appBar: AppBar(
          title: Text('Other'),
        ),
      );
}

@LorTos

@LorTos please file a new issue with this reproduction and the output of flutter doctor.

@Piinks

@Piinks filed a new issue

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

Navigation Menu

Repository navigation

Cupertino ContextMenu improved onTap support #81057

Use case

Proposal

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Cupertino ContextMenu improved onTap support #81057

Description

Use case

Proposal

Activity

umbrellait-andrey-sosnovyy commented on Aug 25, 2022

thipokch commented on Nov 7, 2022

Piinks commented on May 21, 2025

LorTos commented on May 26, 2025

Piinks commented on May 29, 2025

LorTos commented on Jun 3, 2025

github-actions commented on Jun 17, 2025

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions