[DataTable]: Add ability to only select row using checkbox#105123
[DataTable]: Add ability to only select row using checkbox#105123fluttergithubbot merged 2 commits intoflutter:masterfrom
Conversation
| /// tap or long press. | ||
| /// | ||
| /// This value is true by default. | ||
| final bool selectableOnGestures; |
There was a problem hiding this comment.
Is there reference to this in spec? I imagine this would be a mobile/tablet behavior only.
There was a problem hiding this comment.
I didn't find any specs, this is in response to the users in #70755.
There was a problem hiding this comment.
I imagine this would be a mobile/tablet behavior only.
This is for all platforms where we don't expect users to tap the row when onTap or long-press are not implemented but only onSelectChanged, which makes the row tappable with inkwell effect and also affects accessibility (talkback says "double-tap to activate").
onTap == null and onSelectChanged != null wraps label with InkWell.
There was a problem hiding this comment.
Gotcha. Can you please add an explanation for how this interacts with onSelectChanged and onRowLongPress?
There was a problem hiding this comment.
Add an explanation, please take a look.
guidezpl
left a comment
There was a problem hiding this comment.
_ ___ _____ __ __
| | / __|_ _| \/ |
| |_| (_ | | | | |\/| |
|____\___| |_| |_| |_|
…lutter#105123)" This reverts commit 5fe7854.
fixes #70755
minimal code sample
Pre-launch Checklist
///).If you need help, consider asking for advice on the #hackers-new channel on Discord.