Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Start using the user's profile picture in more places #4625

Closed
ThiefMaster opened this issue Sep 9, 2020 · 4 comments · Fixed by #4747
Closed

Start using the user's profile picture in more places #4625

ThiefMaster opened this issue Sep 9, 2020 · 4 comments · Fixed by #4747
Assignees
Projects
Milestone

Comments

@ThiefMaster
Copy link
Member

We now have a proper picture_url for every user which will serve a default icon (colored bg + initial) or a custom picture/avatar. Currently this is only used in the user dashboard, but we can now start using this in other places as well.

Look for avatarBgColor in the code to find those places!

@pferreir
Copy link
Member

pferreir commented Sep 9, 2020

It would be nice to also add the pictures in the list of participants of a meeting (as long as the list is visible, obviously).

@ThiefMaster ThiefMaster added this to the v3 milestone Nov 18, 2020
@ThiefMaster ThiefMaster added this to To do in Release 3.0 via automation Nov 18, 2020
@Leats Leats self-assigned this Dec 14, 2020
@Leats
Copy link
Contributor

Leats commented Dec 16, 2020

It would be nice to also add the pictures in the list of participants of a meeting (as long as the list is visible, obviously).

@pferreir Did you mean this here? Because I'd worry that the profile pictures would be too small.
Bildschirmfoto 2020-12-16 um 09 57 06
Otherwise, maybe in the participant list of a conference?

@pferreir
Copy link
Member

Yeah. We'd probably have to make those bigger...

@ThiefMaster
Copy link
Member Author

not sure how that would look like for larger meetings with more than a handful of participants :/ doing it on the participant list for a conference would be easier for sure since we have a proper table there

@Leats Leats moved this from To do to Awaiting review/merge in Release 3.0 Dec 17, 2020
Release 3.0 automation moved this from Awaiting review/merge to Done Jan 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Release 3.0
  
Done
Development

Successfully merging a pull request may close this issue.

3 participants