immich icon indicating copy to clipboard operation
immich copied to clipboard

feat(mobile): Add Partner Sharing Avatars to Timeline and Info Cards

Open x24git opened this issue 1 year ago • 11 comments
trafficstars

This Feature adds user avatars to the timeline for both partner sharing, and shared albums. The avatar uses the user's avatar image (if one is set) or the server's information is used to make an avatar using the user's assigned color and first letter of their name. (In the screenshot below, the partner's icon is an Orange 'T')

This is an optional configuration that adds a new setting in the Photo Grid section of the settings.

Additionally, storage information is now shown in the image details card. For partner sharing it will show which partner it belongs to. For other images, it provides a text description and existing storage icons if the photo is local, remote, or merged (backed up). This can be useful if the storage icons are hidden.

x24git avatar Aug 27 '24 23:08 x24git

I am working on implementing a similar feature set on the web app to ensure consistent experience

x24git avatar Aug 27 '24 23:08 x24git

Implements https://github.com/immich-app/immich/discussions/7608 (actually for the web but the original PR publisher writes that he will consider a web version)

aviv926 avatar Aug 28 '24 00:08 aviv926

aviv926

Yeah, the plan was to put mobile and web together in one PR but ran into some issues getting it done efficiently on the web side, so I split the work. Should have the web implementation done shortly, and that would implement #7608 almost exactly like you mocked it up.

x24git avatar Aug 28 '24 03:08 x24git

Cleaned up commit history and rebased to main, resolving merge conflicts.

x24git avatar Sep 06 '24 23:09 x24git

PR https://github.com/immich-app/immich/pull/12413 implements the same feature set on web.

x24git avatar Sep 06 '24 23:09 x24git

Just curious, but is this feature still not available? I haven't seen any indication of who uploaded what to a shared album. Would be a nice addition. :)

scrampker avatar Oct 16 '24 02:10 scrampker

Just curious, but is this feature still not available? I haven't seen any indication of who uploaded what to a shared album. Would be a nice addition. :)

@scrampker Just updated the PR to incorporate the changes made in the mobile app as of 1.18. Should be good to merge (pending review).

Just need a maintainer to tag it and then review it :)

x24git avatar Oct 16 '24 08:10 x24git

Hello, thank you for the PR. I just ran testing on this PR. A few points

  • Turn off "Show user thumbnails" in the App Settings doesn't take effect.
  • ~~I would like to have this mechanism disabled by default.~~ Actually, having it on by default is nice.
  • In the asset viewer, putting the users' avatar on the top app bar looks strange. I think having the information inside the detail panel is enough

alextran1502 avatar Oct 18 '24 20:10 alextran1502

Hello, thank you for the PR. I just ran testing on this PR. A few points

  • Turn off "Show user thumbnails" in the App Settings doesn't take effect.
  • ~I would like to have this mechanism disabled by default.~ Actually, having it on by default is nice.
  • In the asset viewer, putting the users' avatar on the top app bar looks strange. I think having the information inside the detail panel is enough

Nice catch on the settings issue, and I do agree user avatar at the top is a bit out of place (since everything else up there is an action). I think there is enough context clues (like missing download and favorite icons) to indicate if an image is being shared.

x24git avatar Oct 18 '24 20:10 x24git

Is a part of this feature set to "add to library?" If someone uploads to a shared, and you want it to be added to your main view. IE if the shared album is deleted or the user removes their assets.

scrampker avatar Oct 19 '24 15:10 scrampker

@scrampker that's entirely unrelated to this PR

bo0tzz avatar Oct 19 '24 15:10 bo0tzz

@x24git Apologies for the lack of update and thanks a lot for the contribution. Can you please rebase it over main and we can check and get this merged?

shenlong-tanwen avatar Mar 31 '25 18:03 shenlong-tanwen

Closing due to stale. We can look into this at a later time

alextran1502 avatar May 08 '25 13:05 alextran1502