DataTable: customizable mouse cursor #123020

Activity

1. darshankawar

   added

   in triagePresently being triaged by the triage teamPresently being triaged by the triage team

   c: new featureNothing broken; request for a new capabilityNothing broken; request for a new capability

   c: proposalA detailed proposal for a change to FlutterA detailed proposal for a change to Flutter

   a: desktopRunning on desktopRunning on desktop

   a: mouseIssues related to using a mouse or mouse supportIssues related to using a mouse or mouse support

   frameworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.

   p: material_uimaterial_ui package in flutter/packagesmaterial_ui package in flutter/packages

   and removed

   in triagePresently being triaged by the triage teamPresently being triaged by the triage team

   on Mar 21, 2023
2. TahaTesser commented on Mar 21, 2023

   

   TahaTesser

   on Mar 21, 2023

   Last edited by TahaTesser

   Contributor

   More actions

   I'm working on this @jpnurmi. Should be an easy fix.

   ❤️React with ❤️1Reacted by J-P Nurmi
3. TahaTesser commented on Mar 21, 2023

   

   TahaTesser

   on Mar 21, 2023

   Contributor

   More actions

   @jpnurmi
   It'd be like this, what do you think?

               mouseCursor: MaterialStateProperty.resolveWith<MouseCursor>((Set<MaterialState> states) {
                 if (states.contains(MaterialState.selected)) {
                   return SystemMouseCursors.copy;
                 }
                 return SystemMouseCursors.contextMenu;
               }),
               cells: <DataCell>[DataCell(Text('Row $index'))],
               selected: selected[index],
               onSelectChanged: (bool? value) {

   Screen.Recording.2023-03-21.at.10.56.33.mov

   🚀React with 🚀1Reacted by J-P Nurmi
4. jpnurmi commented on Mar 21, 2023

   

   jpnurmi

   on Mar 21, 2023

   MemberAuthor

   More actions

   @TahaTesser wow, that was quick! 🚀 looks good! notice that the header may also change the cursor if sortable.

   👍React with 👍1Reacted by Taha Tesser
5. TahaTesser commented on Mar 21, 2023

   

   TahaTesser

   on Mar 21, 2023

   Last edited by TahaTesser

   Contributor

   More actions

   @jpnurmi
   Added cursor customizable for header cell as well.
   I'd appreciate it if you can test my code implementation and provide any feedback on the behavior and naming #123128

   👍React with 👍1Reacted by J-P Nurmi
6. TahaTesser

   self-assigned this

   on Mar 21, 2023
7. TahaTesser

   added this to Nevercodeon Mar 21, 2023
8. github-project-automation

   moved this to To do in Nevercodeon Mar 21, 2023
9. TahaTesser

   moved this from To do to PR submitted in Nevercodeon Mar 21, 2023
10. jpnurmi commented on Mar 21, 2023

    

    jpnurmi

    on Mar 21, 2023

    MemberAuthor

    More actions

    Awesome, will do! 👍

    ❤️React with ❤️1Reacted by Taha Tesser
11. gspencergoog

    added

    P3Issues that are less important to the Flutter projectIssues that are less important to the Flutter project

    on Mar 23, 2023
12. HansMuller

    closed this as completedin #123128on Mar 24, 2023
13. github-project-automation

    moved this from PR submitted to Done (PR merged) in Nevercodeon Mar 24, 2023
14. TahaTesser

    added

    r: fixedIssue is closed as already fixed in a newer versionIssue is closed as already fixed in a newer version

    on Mar 24, 2023
15. github-actions commented on Apr 7, 2023

    

    github-actionsBot

    on Apr 7, 2023

    More 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.
16. github-actions

    locked as resolved and limited conversation to collaborators on Apr 7, 2023