-
Notifications
You must be signed in to change notification settings - Fork 882
Improve default avatar style #14997
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
Comments
Design LGTM 👍🏼 |
@christin I noticed we currently have three sizes for the avatar, and I’m wondering if we should add another size for pages like this one—the template page—where the avatar is quite large (48px). ![]() |
I think aliases will be better so we can stick to a standard set of sizes. |
Should this set of sizes be specific to the component or more broadly defined? For example, would the |
I think component specific makes sense also for consistency we should probably keep the Tailwind naming conventions of "xs","sm","md","lg","xl", etc |
Sounds good, I updated the component naming in Figma accordingly (see component)
Good catch! I created a quick mockup and it seems like the lg size (40px) fits great also in this context |
Related to #14997 - Add a new `Avatar` component based on the [new avatar design](https://www.figma.com/design/WfqIgsTFXN2BscBSSyXWF8/Coder-kit?node-id=711-383&t=xqxOSUk48GvDsjGK-0). - Deprecate existent `Avatar` component.
Just sharing my status. It’s been more work than I initially thought. I didn’t realize how many different places were using the Avatar component with custom styles applied. I’ve managed to get the new design in place, but a few components need extra work to look good with the changes. Chromatic flagged 254 affected snapshots, and I’m reviewing them one by one to ensure everything is in place and looks professional. |
Problem:

the current default avatar is very boring, has no character and can't be distinguished if they are several of them
Solution:
Let's create something more interesting and that represents the initals of the name
See Figma

The text was updated successfully, but these errors were encountered: