Skip to content

Conversation

@SuaYoo
Copy link
Member

@SuaYoo SuaYoo commented Dec 4, 2025

In preparation for #2874

Changes

  • Fixes browser profile page width
  • Use badges in archived item page header
  • Removed "QA Analysis Runs" column from archived items to create space for more critical information
  • Fix upload status not displayed in detail view
  • Fix user chip font

Screenshots

Page Image/video
Archived Item Screenshot 2025-12-04 at 1 54 53 PM
Archived Item (no collections) Screenshot 2025-12-04 at 1 55 33 PM
Archived Item (reviewed) Screenshot 2025-12-04 at 1 56 03 PM
Archived item (upload) Screenshot 2025-12-04 at 1 57 14 PM

@SuaYoo SuaYoo marked this pull request as ready for review December 4, 2025 22:01
@SuaYoo SuaYoo requested review from emma-sg, ikreymer and tw4l December 4, 2025 22:01
Copy link
Member

@tw4l tw4l left a comment

Choose a reason for hiding this comment

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

Looks good!

I wonder if it would be more helpful to have the "Reviewed" archived item page header badge have the rating in the text instead of just on hover (it also appears to be just the number in the hover text rather than the appropriate text string from the review status enum).

So, e.g., instead of Reviewed, the badge could read Reviewed: Excellent or even QA Rating: Excellent to match the sort option in the archived items list.

@SuaYoo
Copy link
Member Author

SuaYoo commented Dec 8, 2025

I wonder if it would be more helpful to have the "Reviewed" archived item page header badge have the rating in the text instead of just on hover

Good point! Updated:

Screenshot 2025-12-08 at 10 02 24 AM Screenshot 2025-12-08 at 10 02 40 AM Screenshot 2025-12-08 at 10 03 05 AM Screenshot 2025-12-08 at 10 03 14 AM

IMO there's enough context with the color and icon to show the "QA Rating" label only in the tooltip:

Screenshot 2025-12-08 at 10 03 41 AM

Could be convinced otherwise, though.

@tw4l
Copy link
Member

tw4l commented Dec 8, 2025

Good point! Updated...

IMO there's enough context with the color and icon to show the "QA Rating" label only in the tooltip...

Yes I think this looks great! I agree that there's enough context there already that it shouldn't be confusing this way.

Copy link
Member

@emma-sg emma-sg left a comment

Choose a reason for hiding this comment

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

Nice, looks great!

@SuaYoo SuaYoo merged commit f396c0b into main Dec 8, 2025
29 checks passed
@SuaYoo SuaYoo deleted the frontend-ui-consistency-fixes branch December 8, 2025 19:11
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.

4 participants