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: update jetbrains sponsor logo #536

Merged
merged 2 commits into from
May 29, 2024

Conversation

niklas-wortmann
Copy link
Contributor

We updated our logo for JetBrains IDEs. I hope it is ok that I increased the size, it kinda looked weird next to the others. Hope it wasn't associated with a certain sponsor package or something. If so, I am more than happy to revert that change

Copy link

stackblitz bot commented May 16, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copy link

vercel bot commented May 16, 2024

Someone is attempting to deploy a commit to the pnpm Team on Vercel.

A member of the Team first needs to authorize it.

@zkochan
Copy link
Member

zkochan commented May 16, 2024

This will only look good in the dark theme (which is off by default).

@niklas-wortmann
Copy link
Contributor Author

@zkochan that's a good point, I will test a couple different logo variation. Thanks!

Copy link

vercel bot commented May 17, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
pnpm-io ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 29, 2024 7:04pm

@zkochan
Copy link
Member

zkochan commented May 29, 2024

How did you manage to make the logo change its color in the different theme?

@zkochan zkochan merged commit af9b3d3 into pnpm:main May 29, 2024
3 checks passed
@niklas-wortmann
Copy link
Contributor Author

svg has support for the style attribute so I just ship some basic styles with the SVG
image

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