Skip to content

Conversation

@SuaYoo
Copy link
Member

@SuaYoo SuaYoo commented Jan 30, 2024

Resolves #1483

@Shrinks99 while we're still settling on data table variations, I added a shadow to collection and browser profile items to match workflows more for now.

Changes

  • Updates browser profile list styles to match other data table styles
  • Makes entire collection item clickable
  • Refactors row click area to fix text overflow

Manual testing

  1. Log in and go to browser profiles. Verify items can be clicked to view details and menu items work
  2. Go to Collections. Verify items can be clicked to view details and menu items work

Screenshots

Page Image/video
Browser Profiles Screenshot 2024-01-30 at 5 05 21 PM
Collections Screenshot 2024-01-30 at 4 57 03 PM

@SuaYoo SuaYoo changed the base branch from main to frontend-checkbox-list--crawl-list January 30, 2024 22:51
Base automatically changed from frontend-checkbox-list--crawl-list to main January 31, 2024 00:08
@SuaYoo SuaYoo force-pushed the frontend-use-data-tables branch 2 times, most recently from 2b60629 to e4b31b6 Compare January 31, 2024 00:55
@SuaYoo SuaYoo force-pushed the frontend-use-data-tables branch from e4b31b6 to 2ee2888 Compare January 31, 2024 00:59
@SuaYoo SuaYoo marked this pull request as ready for review January 31, 2024 01:03
Copy link
Collaborator

@Shrinks99 Shrinks99 left a comment

Choose a reason for hiding this comment

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

while we're still settling on data table variations, I added a shadow to collection and browser profile items to match workflows more for now.

Makes sense!

One small suggestion, then good from me!

@SuaYoo SuaYoo merged commit 79645b6 into main Jan 31, 2024
@SuaYoo SuaYoo deleted the frontend-use-data-tables branch January 31, 2024 03:46
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.

Refactor list components to use table component

4 participants