-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Closed
Closed
Copy link
Labels
Design Systems StudyIssues identified during the custom design system study.Issues identified during the custom design system study.P2Important issues not at the top of the work listImportant issues not at the top of the work listc: proposalA detailed proposal for a change to FlutterA detailed proposal for a change to Flutterf: material designflutter/packages/flutter/material repository.flutter/packages/flutter/material repository.frameworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.r: fixedIssue is closed as already fixed in a newer versionIssue is closed as already fixed in a newer versionteam-designOwned by Design Languages teamOwned by Design Languages teamtriaged-designTriaged by Design Languages teamTriaged by Design Languages team
Description
Is there an existing issue for this?
- I have searched the existing issues
- I have read the guide to filing a bug
Use case
Currently showMenu (and classes that use it, such as PopupMenuButton) has a hard-coded 300ms animation when the menu opens or closes. This cannot be modified by any means I have discovered.
This is inconsistent with MenuAnchor-based menus, which have no animation at all, and particularly for right-click context menus it feels far too slow.
Proposal
Looking at the code, it would be fairly straightforward to allow the duration to be overridden via an optional parameter to showMenu and then passed through to the _PopupMenuRoute that's created to service the call. Ideally, PopupMenuButton and friends would also accept that optional parameter and pass it through to showMenu.
alexvoina
Metadata
Metadata
Assignees
Labels
Design Systems StudyIssues identified during the custom design system study.Issues identified during the custom design system study.P2Important issues not at the top of the work listImportant issues not at the top of the work listc: proposalA detailed proposal for a change to FlutterA detailed proposal for a change to Flutterf: material designflutter/packages/flutter/material repository.flutter/packages/flutter/material repository.frameworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.r: fixedIssue is closed as already fixed in a newer versionIssue is closed as already fixed in a newer versionteam-designOwned by Design Languages teamOwned by Design Languages teamtriaged-designTriaged by Design Languages teamTriaged by Design Languages team
Type
Projects
Status
Done (PR merged)
Status
Done