Skip to content

chore(web): minor UX improvements of "view asset owners" feature#24319

Merged
alextran1502 merged 9 commits intomainfrom
feature/improve-owner-label
Dec 1, 2025
Merged

chore(web): minor UX improvements of "view asset owners" feature#24319
alextran1502 merged 9 commits intomainfrom
feature/improve-owner-label

Conversation

@idubnori
Copy link
Copy Markdown
Collaborator

@idubnori idubnori commented Dec 1, 2025

Description

Details: please look at the ScreenShots section below

  • Move the button into the kebab menu
    • I think "view asset owners" feature is used less frequently compared to other buttons
    • also, display description text allows users to understand the function immidiatly.
  • Persist the feature's enabled/disabled state in local storage
  • Switch icons based on feature enabled/disabled

base PR: #21171

Fixes # (issue)

  • N/A

How Has This Been Tested?

Unit Tests

  • Persist the feature's enabled/disabled state in local storage

Manually Tests on Browser

  • The enabled/disabled status of features is persisted and retained in local storage. (keep the default is disabled)
  • Icons switch based on the enabled/disabled status of features.

Screenshots (if appropriate)

screen-recording-immich-14.mp4

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if applicable
  • I have no unrelated changes in the PR.
  • I have confirmed that any new dependencies are strictly necessary.
  • I have written tests for new code (if applicable)
  • I have followed naming conventions/patterns in the surrounding code
  • All code in src/services/ uses repositories implementations for database calls, filesystem operations, etc.
  • All code in src/repositories/ is pretty basic/simple and does not have any immich specific logic (that belongs in src/services/)

Please describe to which degree, if any, an LLM was used in creating this pull request.

most of the implementation, with my review

@idubnori idubnori changed the title chore(web): minor UX improvement of "view asset owners" feature chore(web): minor UX improvements of "view asset owners" feature Dec 1, 2025
@idubnori idubnori marked this pull request as ready for review December 1, 2025 16:11
@alextran1502 alextran1502 merged commit d8ca210 into main Dec 1, 2025
65 of 67 checks passed
@alextran1502 alextran1502 deleted the feature/improve-owner-label branch December 1, 2025 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants