Skip to content

Conversation

@lukashass
Copy link
Contributor

@lukashass lukashass commented Jul 14, 2025

Description

Fixes button sizes. See screenshots.

How Has This Been Tested?

  • Manual test

Screenshots (if appropriate)

before after
image image

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/)

@jrasm91
Copy link
Member

jrasm91 commented Jul 14, 2025

Is there a reason that not all of the buttons have shape="round"? I feel like that would look better, personally.

@lukashass
Copy link
Contributor Author

lukashass commented Jul 14, 2025

Is there a reason that not all of the buttons have shape="round"? I feel like that would look better, personally.

Looks like it was just missing, added it and also increased the unassigned-icon size to match the buttons.

@jrasm91 jrasm91 merged commit ccd0c35 into immich-app:main Jul 14, 2025
49 checks passed
@lukashass lukashass deleted the fix-person-side-panel-buttons branch July 17, 2025 13:46
ollioddi pushed a commit to ollioddi/immich that referenced this pull request Aug 11, 2025
* fix(web): adjust button size in person side panel

* round edit button
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