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

Use a generated SVG logo #175

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

trexfeathers
Copy link
Contributor

🚀 Pull Request

Use a generated SVG logo instead of the current one. Please tweak this however you like to get the logo you really want. And note that until this afternoon I had never used Tephi so if I've done something heinous please call me out on it!

Description

  • Avoids pixellation - infinitely scalable
  • Tephi logo generated via Tephi is kinda neat
  • Using a script means that changes will be more easily human-readable

Comment on lines -137 to -138
# docs. This file should be a Windows icon file (.ico) being 16x16 or 32x32
# pixels large.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This hasn't been true for years for most web browsers. The Iris Readthedocs page uses an SVG and that works just fine.

@trexfeathers
Copy link
Contributor Author

Gonna try to get it back down to the original size.

@trexfeathers trexfeathers marked this pull request as draft November 25, 2024 16:04
@trexfeathers
Copy link
Contributor Author

Gonna try to get it back down to the original size.

OK regardless of pixel size it seems to still render pretty big in RTD. There is probably some config we can put in conf.py but I've realised I should be leaving it to the reviewer to decide what they would prefer.

image

@trexfeathers trexfeathers marked this pull request as ready for review November 25, 2024 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant