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

Replace project avatars #403

Merged
merged 7 commits into from
Aug 15, 2024
Merged

Replace project avatars #403

merged 7 commits into from
Aug 15, 2024

Conversation

agjohnson
Copy link
Contributor

@agjohnson agjohnson commented Jun 29, 2024

  • Fix URL for web component styles and avatar image
  • Create a 100x100px image, mapped to our current color scheme colors on the dashboard. This creates a lot of nice variance in colors and matches the look and feel in addition.
  • Resize to something easy for bitwise hashing math, 128x128?
  • Find the right pattern for avoiding inline styles and background positioning
  • Fix positioning on the project header, it's out of place now

@agjohnson
Copy link
Contributor Author

Image

@agjohnson
Copy link
Contributor Author

agjohnson commented Jul 3, 2024

I worked through patterns for attributes, using a static URL for the image, updated the underlying sprite, and fixed an issue on the project header.

With the new sprite, the dashboard looks like this now:

image

The image is a palette mapped image, using colors from the FUI base color scheme. We can also tweak the sprite more though. Here's an example using fewer colors and more pixels in the avatar image:

image

@agjohnson agjohnson marked this pull request as ready for review July 3, 2024 06:11
@agjohnson agjohnson requested a review from a team as a code owner July 3, 2024 06:11
@agjohnson
Copy link
Contributor Author

I like the look of the 8x8 pixel avatar, but would say the 4x4 looks more like it is a generated icon. The color limited color scheme works much nicer too, all the colors are prominent in the dashboard already.

I think my favorite is 4x4 pixels, limited FUI color palette:

image

Copy link
Member

@humitos humitos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. The only thing I would change is the background image used. I'd propose to generate one ourselves in some random way by taking a color palette from https://colorhunt.co/ that looks nice.

@agjohnson
Copy link
Contributor Author

I am already using the same color palette as FUI. What is the concern around generating an image vs processing a photo?

Copy link
Member

@ericholscher ericholscher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

@agjohnson agjohnson merged commit c4d40d6 into main Aug 15, 2024
4 checks passed
@agjohnson agjohnson deleted the agj/new-avatar branch August 15, 2024 01:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace Dicebear with static generated images
3 participants