Skip to content

Conversation

@sumebrius
Copy link
Contributor

If a repo was defined for the readthedocs theme, but the repo wasn't a predefined provider (GitHub, Bitbucket, or GitLab), no repo link would be rendered.

This adds a fallback generic git repo link.

Fixes #3259

@oprypin
Copy link
Contributor

oprypin commented Oct 20, 2023

Hmm the logo literally just says "git".. which may be controversial seeing as it doesn't even have to be a git repo.
Usually the logo "fa-git-alt" is a better fit but seems it's not present in our old version of font-awesome

@sumebrius
Copy link
Contributor Author

Yeah, the font-awesome 4.7 set is a little more limited than I was expecting.

code-fork might be a more VCS-agnostic choice?

@oprypin
Copy link
Contributor

oprypin commented Oct 20, 2023

Yeah that would be great 👍

@sumebrius
Copy link
Contributor Author

Done!

Copy link
Member

@ultrabug ultrabug left a comment

Choose a reason for hiding this comment

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

Thanks!

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.

Repo Link missing in readthedocs theme if custom repo_name used

3 participants