Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • Briar Desktop Briar Desktop
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 177
    • Issues 177
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 10
    • Merge requests 10
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • briar
  • Briar DesktopBriar Desktop
  • Merge requests
  • !175

Don't use text PointerIcon on buttons in text fields

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Nico requested to merge 256-hovering into main Feb 18, 2022
  • Overview 1
  • Commits 1
  • Pipelines 4
  • Changes 3

Thanks to the research of Paul, I could quickly find the right modifier to change the mouse appearance upon hovering buttons.

For most buttons we stay with the default PointerIcon, just inside text fields it showed the text PointerIcon. With this commit the default PointerIcon is shown consistently across all buttons.

In a later MR we could switch all buttons to show the hand icon, but I wouldn't do it because later Compose versions might do this automatically and the APIs used here are experimental.

Closes #256 (closed)

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: 256-hovering