Skip to content

[PB-5663] feat: favorite indicator in list view - #2044

Open
victor-ferro wants to merge 6 commits into
PB-5662-toolbar-favoritefrom
PB-5663-favorite-indicator
Open

[PB-5663] feat: favorite indicator in list view#2044
victor-ferro wants to merge 6 commits into
PB-5662-toolbar-favoritefrom
PB-5663-favorite-indicator

Conversation

@victor-ferro

@victor-ferro victor-ferro commented Jul 14, 2026

Copy link
Copy Markdown

Description

  • Star indicator in the list view rows (DriveExplorerListItem), at the end of the name column: filled yellow star always visible for favorited items, outline star on row hover for the rest
  • Clicking the star toggles the favorite state via the existing toggleFavoriteThunk, without selecting or opening the row (stopPropagation)
  • Hidden in Trash, workspaces and grid view; no interference with selection, the sharing badge or row actions

Related Issues

Relates to PB-5663

Related Pull Requests

Checklist

  • Changes have been tested locally.
  • Unit tests have been written or updated as necessary.
  • The code adheres to the repository's coding standards.
  • Relevant documentation has been added or updated.
  • No new warnings or errors have been introduced.
  • SonarCloud issues have been reviewed and addressed.
  • QA Passed

Testing Process

  • Manual testing: filled/outline star states, hover, click without selecting, grid view clean.

Additional Notes

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 15, 2026

Copy link
Copy Markdown

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8a3e47c
Status: ✅  Deploy successful!
Preview URL: https://fe8fd5fb.drive-web.pages.dev
Branch Preview URL: https://pb-5663-favorite-indicator.drive-web.pages.dev

View logs

@victor-ferro victor-ferro self-assigned this Jul 24, 2026

@larryrider larryrider left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

check sonar cloud issue

@sonarqubecloud

Copy link
Copy Markdown

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