Skip to content

Expose onSecondaryTap in InkWell #118622

New issue
New issue
Closed
#119058
Closed
Expose onSecondaryTap in InkWell#118622
#119058
Assignees
Renzo-Olivares
Labels
P3Issues that are less important to the Flutter projecta: desktopRunning on desktopc: new featureNothing broken; request for a new capabilityc: proposalA detailed proposal for a change to Flutterf: material designflutter/packages/flutter/material repository.frameworkflutter/packages/flutter repository. See also f: labels.r: fixedIssue is closed as already fixed in a newer version
@mgenware

Description

@mgenware
mgenware
opened on Jan 17, 2023

InkWell is great. I like the opt-in hover effect. Right now InkWell doesn't have onSecondaryTap exposed. It has onLongPress, which is only useful on mobile. It would be weird to ask users to long press with mouse left button instead of using the onSecondaryTap on desktop to trigger a context menu.

flutter/packages/flutter/lib/src/material/ink_well.dart

Lines 1236 to 1244 in cc7845e

onTapDown: enabled ? handleTapDown : null,
onTapUp: enabled ? handleTapUp : null,
onTap: enabled ? handleTap : null,
onTapCancel: enabled ? handleTapCancel : null,
onDoubleTap: widget.onDoubleTap != null ? handleDoubleTap : null,
onLongPress: widget.onLongPress != null ? handleLongPress : null,
behavior: HitTestBehavior.opaque,
excludeFromSemantics: true,
child: widget.child,

Metadata

Metadata

Assignees

Labels

P3Issues that are less important to the Flutter projecta: desktopRunning on desktopc: new featureNothing broken; request for a new capabilityc: proposalA detailed proposal for a change to Flutterf: material designflutter/packages/flutter/material repository.frameworkflutter/packages/flutter repository. See also f: labels.r: fixedIssue is closed as already fixed in a newer version

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions