server icon indicating copy to clipboard operation
server copied to clipboard

Avatar handling

Open tobiasKaminsky opened this issue 5 years ago • 8 comments

  • [ ] Android @tobiasKaminsky https://github.com/nextcloud/android/pull/7098
  • [ ] Desktop @er-vin
  • [ ] iOS @marinofaggiana
  • [ ] Server @nextcloud/server-triage

As discussed with @jancborchardt: image

  • share links:
    • primary background color (currently it is always nc blue)
    • fitting icon color
  • email, others with access, projects, internal link:
    • #ededed as background (always, independent from theming)
    • black icon
  • 1:1 conversation:
    • user avatar
    • fallback: primary background, talk icon
  • group conversation:
    • primary background, talk icon
  • share to group:
    • primary background, multiple user icon
  • federated share:
    • user avatar
    • fallback: 2 initials with computed color
  • circle:
    • #ededed
    • black circle on front, with 1,2, dots to reflect "permission"
    • name (type)
    • icon smaller as like "internal shares"

Grey is "ededed" in light mode with black icon, "#222" in dark mode, with white icon

tobiasKaminsky avatar Oct 15 '20 09:10 tobiasKaminsky

Currently there is no way to distinguish share to a 1:1 conversation or to a group, right? @nickvergessen

tobiasKaminsky avatar Oct 15 '20 10:10 tobiasKaminsky

So server is already like that, right?

skjnldsv avatar Oct 15 '20 10:10 skjnldsv

So server is already like that, right?

Well, server does not allow Talk to provide an icon and forces the first letter of whatever the display name is

Currently there is no way to distinguish share to a 1:1 conversation or to a group, right?

Well the room has this info, the question is how an app can provide it the sharing front end or how apps can do things in the sharing to define an icon.

group conversation:

  • primary background, talk icon

It should currently be the public/group icon on grey background like in talk, and in the next versions we will add the option to upload an image for a conversation and in that case it should use that icon

nickvergessen avatar Oct 15 '20 10:10 nickvergessen

So server is already like that, right?

No: email is primary color, but should be ededed/2222 others with access / internal is wrong background color, should also be ededed/222 with black/white icon

tobiasKaminsky avatar Oct 15 '20 10:10 tobiasKaminsky

This is current new state on Android: image

Missing

  • [ ] distinguish between types of circle
  • [ ] distinguish between 1:1 / group conversation

tobiasKaminsky avatar Oct 15 '20 10:10 tobiasKaminsky

Update on Android: image image

tobiasKaminsky avatar Oct 15 '20 11:10 tobiasKaminsky

@jancborchardt for confirmation

tobiasKaminsky avatar Oct 15 '20 13:10 tobiasKaminsky

Yep, looks great! :) Nice work

jancborchardt avatar Oct 15 '20 15:10 jancborchardt