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

Better logo in dark mode #33799

Closed
wants to merge 2 commits into from
Closed

Conversation

mwarkentin
Copy link
Member

@mwarkentin mwarkentin commented Apr 20, 2022

This uses Github's support for specifying which theme an image should be shown to.

Before:

image

After:

image

@mwarkentin mwarkentin enabled auto-merge (squash) April 20, 2022 18:25
Copy link
Member

@billyvg billyvg left a comment

Choose a reason for hiding this comment

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

Oh neat

@chadwhitacre
Copy link
Member

There was some attempt at this in the SDKs that I think was inconclusive. @bruno-garcia @AbhiPrasad Do you remember? I think it was a third-party contribution.

@untitaker
Copy link
Member

getsentry/sentry-python#1342

@mwarkentin mwarkentin disabled auto-merge April 20, 2022 18:42
@mwarkentin
Copy link
Member Author

mwarkentin commented Apr 20, 2022

@untitaker @chadwhitacre Thanks! Do we render the docs for this repo anywhere else that this would be a concern?

Edit: Looks like we also publish sentry to pypi, so this would probably be an issue there too.

@mwarkentin
Copy link
Member Author

Closing until there's better support in other places that these docs are published.

@mwarkentin mwarkentin closed this Apr 20, 2022
@mwarkentin mwarkentin deleted the add-dark-mode-logo-to-readme branch April 20, 2022 19:02
@github-actions github-actions bot locked and limited conversation to collaborators May 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants