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

docs and readme: add the new jj logo #5554

Merged
merged 1 commit into from
Feb 7, 2025
Merged

docs and readme: add the new jj logo #5554

merged 1 commit into from
Feb 7, 2025

Conversation

ilyagr
Copy link
Contributor

@ilyagr ilyagr commented Feb 2, 2025

I modified the logo slightly: the corners are rounded, the birds are larger.

This also adds the 400KB full-resolution logo to the repo. I think it's worth it, but alternatively we could ass it to https://github.com/jj-vcs/jj-vcs.github.io instead.

@ilyagr ilyagr marked this pull request as ready for review February 2, 2025 04:04
@ilyagr ilyagr force-pushed the ig/readme-logo branch 4 times, most recently from 159ce94 to ef008c3 Compare February 2, 2025 04:12
@ilyagr ilyagr marked this pull request as draft February 2, 2025 04:13
@ilyagr ilyagr force-pushed the ig/readme-logo branch 4 times, most recently from 7b5763c to 3f09da0 Compare February 2, 2025 04:17
@ilyagr ilyagr marked this pull request as ready for review February 2, 2025 04:18
@ilyagr
Copy link
Contributor Author

ilyagr commented Feb 7, 2025

Thanks!

I modified the picture slightly by compressing it with pngquant. I cannot tell the difference, you can see it (for now) at ilyagr@readme-log#diff-10f1fa4e6bf66cac7012a113fa83ca97488b80d0060b506c337a48a5b514df92 (the second diff).

@ilyagr
Copy link
Contributor Author

ilyagr commented Feb 7, 2025

I might also decrease the resolution by half, to get a 432x432 image that's 50KB instead of 160KB. That should load quickly in people's browsers. (done)

@ilyagr ilyagr force-pushed the ig/readme-logo branch 2 times, most recently from 6875fbb to fd6df5a Compare February 7, 2025 05:39
It's half-sized (432x432 instead of 864x864) for size and
compressed with `pngquant` (difference not visible with
my eyes).
@ilyagr ilyagr added this pull request to the merge queue Feb 7, 2025
Merged via the queue into main with commit 1d95bd6 Feb 7, 2025
38 checks passed
@ilyagr ilyagr deleted the ig/readme-logo branch February 7, 2025 20:54
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.

2 participants