immich-app/immich
1
0
Fork 0
mirror of https://github.com/immich-app/immich.git synced 2026-07-16 05:34:30 +03:00
Code Issues 783 Packages Projects Releases 22 Wiki Activity

[PR #24675] feat(mobile): show asset owner #17962

New Issue
Open
opened 2026-02-05 16:30:18 +03:00 by OVERLORD · 0 comments
OVERLORD commented 2026-02-05 16:30:18 +03:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/immich-app/immich/pull/24675
Author: @idubnori
Created: 12/18/2025
Status: 🔄 Open

Base: main ← Head: feature/mobile-view-asset-owner

📝 Commits (8)

  • f13a5ba feat(mobile): add option to show asset owner name in asset list
  • 9262f69 feat(mobile): improve owner name display in thumbnail tile
  • 85086cd revert: changes...
  • cc3307d refactor(mobile): enhance thumbnail layout and owner name display
  • ea37b12 refactor(mobile): simplify owner name retrieval logic in asset tile widget
  • 5050f8f refactor(mobile): adjust padding and font weight in thumbnail tile widget
  • 501617b refactor(mobile): optimize padding and metrics handling in thumbnail tile widget
  • c90c7a5 refactor(mobile): unify padding metrics in thumbnail tile widget

📊 Changes

7 files changed (+209 additions, -76 deletions)

View changed files

📝 i18n/en.json (+1 -0)
📝 mobile/lib/domain/models/setting.model.dart (+2 -1)
📝 mobile/lib/domain/models/store.model.dart (+1 -0)
📝 mobile/lib/presentation/widgets/images/thumbnail_tile.widget.dart (+167 -74)
📝 mobile/lib/presentation/widgets/timeline/fixed/segment.model.dart (+27 -0)
📝 mobile/lib/services/app_settings.service.dart (+2 -1)
📝 mobile/lib/widgets/settings/asset_list_settings/asset_list_settings.dart (+9 -0)

📄 Description

Description

How Has This Been Tested?

  • Test A
  • Test B

Screenshots (if appropriate)

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.

...

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/immich-app/immich/pull/24675 **Author:** [@idubnori](https://github.com/idubnori) **Created:** 12/18/2025 **Status:** 🔄 Open **Base:** `main` ← **Head:** `feature/mobile-view-asset-owner` --- ### 📝 Commits (8) - [`f13a5ba`](https://github.com/immich-app/immich/commit/f13a5ba41854fd9ba2fb23d756d942c00329f123) feat(mobile): add option to show asset owner name in asset list - [`9262f69`](https://github.com/immich-app/immich/commit/9262f695197d48837c6d03967dd07b44d91b9501) feat(mobile): improve owner name display in thumbnail tile - [`85086cd`](https://github.com/immich-app/immich/commit/85086cdb02f5a50266e7ee87ba3c72d7f7c0537a) revert: changes... - [`cc3307d`](https://github.com/immich-app/immich/commit/cc3307d335c877746a753771398e955f9be3bbaf) refactor(mobile): enhance thumbnail layout and owner name display - [`ea37b12`](https://github.com/immich-app/immich/commit/ea37b12b3902d8bc74e593fc0f267d8e756c06cc) refactor(mobile): simplify owner name retrieval logic in asset tile widget - [`5050f8f`](https://github.com/immich-app/immich/commit/5050f8fbb4647a63e5ee2ecd2d622c917e1317e1) refactor(mobile): adjust padding and font weight in thumbnail tile widget - [`501617b`](https://github.com/immich-app/immich/commit/501617ba93ec90768c45dbdc434a3149baaad9eb) refactor(mobile): optimize padding and metrics handling in thumbnail tile widget - [`c90c7a5`](https://github.com/immich-app/immich/commit/c90c7a56cea7b41e9706e4a6143466afd98f9095) refactor(mobile): unify padding metrics in thumbnail tile widget ### 📊 Changes **7 files changed** (+209 additions, -76 deletions) <details> <summary>View changed files</summary> 📝 `i18n/en.json` (+1 -0) 📝 `mobile/lib/domain/models/setting.model.dart` (+2 -1) 📝 `mobile/lib/domain/models/store.model.dart` (+1 -0) 📝 `mobile/lib/presentation/widgets/images/thumbnail_tile.widget.dart` (+167 -74) 📝 `mobile/lib/presentation/widgets/timeline/fixed/segment.model.dart` (+27 -0) 📝 `mobile/lib/services/app_settings.service.dart` (+2 -1) 📝 `mobile/lib/widgets/settings/asset_list_settings/asset_list_settings.dart` (+9 -0) </details> ### 📄 Description ## Description - mobile side of https://github.com/immich-app/immich/pull/21171 <!--- Describe your changes in detail --> <!--- Why is this change required? What problem does it solve? --> <!--- If it fixes an open issue, please link to the issue here. --> ## How Has This Been Tested? <!-- Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration --> - [ ] Test A - [ ] Test B <details><summary><h2>Screenshots (if appropriate)</h2></summary> <!-- Images go below this line. --> </details> <!-- API endpoint changes (if relevant) ## API Changes The `/api/something` endpoint is now `/api/something-else` --> ## 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. ... --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 16:30:18 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
accessibility
changelog:enhancement
changelog:security
changelog:skip
changelog:translation
cli
date-time
dependencies
documentation
external-library
format
good first issue
mobile-beta
mobile-beta
mobile-beta
needs-answer
nice to have
pull-request
Mirrored from GitHub Pull Request
 sharing
tech-debt
📱mobile
🖥️web
🗄️server
🧠machine-learning
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#17962
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?