Skip to content

Improve mouse navigation and enable CRTL+Click#685

Merged
rpbritton merged 1 commit intomainfrom
allow-ctrl-click
Apr 6, 2026
Merged

Improve mouse navigation and enable CRTL+Click#685
rpbritton merged 1 commit intomainfrom
allow-ctrl-click

Conversation

@mclemenceau
Copy link
Copy Markdown
Collaborator

@mclemenceau mclemenceau commented Apr 3, 2026

AI Use Disclosure: This change was authored with assistance from AI. The code was reviewed, tested locally, and validated as much as possible.

Description

This pull request refactors how navigation is handled for artifact cards and rows in the dashboard.
Instead of using GestureDetector or InkWell directly, it introduces and utilizes a unified NavigableLink widget.
Additionally, it updates the internal implementation of NavigableLink to use InkWell

This change allows better navigation and use of the mouse to open several card or row in new tabs.

Resolved issues

Documentation

N/A

Web service API changes

N/A

Tests

  • Built and tested locally successfully
  • Tested with Card view and List View
  • Test Method:
    • on both snaps and images tabs
    • click on several items and then CRTL+CLICK on several items
      • click takes you to a new page
      • ctrl+click open a new tab
    • Switch to card view
    • click on several items and then CRTL+CLICK on several items
      • click takes you to a new page
      • ctrl+click open a new tab

Copilot AI review requested due to automatic review settings April 3, 2026 21:44
@mclemenceau mclemenceau requested review from rpbritton and removed request for Copilot April 3, 2026 21:46
Copy link
Copy Markdown
Member

@utkarsh2102 utkarsh2102 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

big improvement! +1

Copy link
Copy Markdown
Collaborator

@rpbritton rpbritton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, tested and works great, much needed improvement! Thanks for the contribution!

@rpbritton rpbritton merged commit 970d951 into main Apr 6, 2026
11 checks passed
@rpbritton rpbritton deleted the allow-ctrl-click branch April 6, 2026 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants