Skip to content

site-root: add Radicle logo #1992

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

Merged
merged 1 commit into from
Apr 15, 2025
Merged

site-root: add Radicle logo #1992

merged 1 commit into from
Apr 15, 2025

Conversation

lorenzleutgeb
Copy link
Contributor

@lorenzleutgeb lorenzleutgeb commented Apr 14, 2025

Changes

Adds the logo of the Radicle to the site-root.

Context

Radicle is an open source, peer-to-peer code collaboration stack built on Git. Unlike centralized code hosting platforms, there is no single entity controlling the network. Repositories are replicated across peers in a decentralized manner, and users are in full control of their data and workflow.

Ecosystem: Radicle supported the development of GitOxide, and supports libgit2 and git2 via Drips. There was a talk about Radicle at Git Merge 2024. Radicle is committed to Git and builds heavily on it.

@lorenzleutgeb lorenzleutgeb marked this pull request as draft April 14, 2025 11:02
Copy link
Member

@dscho dscho left a comment

Choose a reason for hiding this comment

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

Looks reasonable. @To1ne what's your take? I could imagine that you didn't so much think of self-hosting when you started that list, right?

@To1ne
Copy link
Collaborator

To1ne commented Apr 14, 2025

@lorenzleutgeb Wait, I was also just also adding this logo. What a coincidence. Awesome.

Unfortunately, the text doesn't work in dark mode:

image

You should be using the default color palette:

  • #aaaaaa for the eyes
  • #999999 for the body
  • #666666 for the horizontal line and the arms
  • #333333 for the text

Then all colors should adapt to dark mode automatically.

And while at it, could you please use bold text, I think it looks better (below is the version I had in my branch):

image

Here's my version of all.svg:

all

@To1ne
Copy link
Collaborator

To1ne commented Apr 14, 2025

Looks reasonable. @To1ne what's your take? I could imagine that you didn't so much think of self-hosting when you started that list, right?

I think it's fine to add this. I didn't think about it, but it's a viable solution.

Shall we then also add https://git.zx2c4.com/cgit/about/ ?

@dscho
Copy link
Member

dscho commented Apr 14, 2025

Shall we then also add https://git.zx2c4.com/cgit/about/ ?

@To1ne Did you think about https://git.zx2c4.com/ instead? But that's not a git forge for the general public...

@lorenzleutgeb lorenzleutgeb marked this pull request as ready for review April 14, 2025 19:28
@lorenzleutgeb
Copy link
Contributor Author

Sorry for the back-and-forth. After some discussion internal to the Radicle team I decided to switch to the pixel-style version. Please reconsider.

Copy link
Member

@dscho dscho left a comment

Choose a reason for hiding this comment

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

Still looks good to me. @To1ne?

@To1ne
Copy link
Collaborator

To1ne commented Apr 15, 2025

Sorry for the back-and-forth. After some discussion internal to the Radicle team I decided to switch to the pixel-style version. Please reconsider.

@lorenzleutgeb I like it.

@dscho I approve!

@dscho dscho merged commit b520d50 into git:gh-pages Apr 15, 2025
1 check passed
@dscho
Copy link
Member

dscho commented Apr 15, 2025

@lorenzleutgeb unfortunately there is no incremental deployment with GitHub Pages, therefore the deployment will take ~5 minutes.

@dscho
Copy link
Member

dscho commented Apr 15, 2025

It's alive:

image

@lorenzleutgeb
Copy link
Contributor Author

Radicle team is very happy and thankful 😄 Also, thanks for the straightforward merge!

@To1ne
Copy link
Collaborator

To1ne commented Apr 15, 2025

I've noticed https://bsky.app/profile/radicle.xyz/post/3lmtnyd5zas2v

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.

3 participants