DataTable: missing secondary tap callback

[jpnurmi]

Use case

Showing context menus on desktop platforms.

Currently, DataCell has onTap, onTapDown, onTapCancel, onLongPress, and onDoubleTap whereas DataRow only has a onLongPress gesture callback.

Proposal

Add onSecondaryTap(Down) to both DataRow and DataCell for showing row- and cell-specific context menus.

[darrenaustin]

cc @justinmc

[justinmc]

It would be nice if we could use composition to get these behaviors, but it seems impractical here, especially because DataCell and DataRow are not widgets. So, I think adding onSecondaryTap and onSecondaryTapDown to DataRow and DataCell makes sense.

Currently, DataCell uses Inkwell's GestureDetector under the hood:

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

Line 870 in d1436d1

label = InkWell(

And Inkwell doesn't have an onSecondaryTap parameter, I'm not sure why. We would need to either add it or wrap it in another GestureDetector there.

[github-actions]

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.