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

add github stars #139

Merged
merged 6 commits into from
Dec 5, 2023
Merged

add github stars #139

merged 6 commits into from
Dec 5, 2023

Conversation

bendichter
Copy link
Collaborator

@bendichter bendichter commented Nov 5, 2023

I think this is a nice unbaissed way for us to indicate the popularity of each of these tools. What does everyone think?

@bendichter bendichter requested a review from oruebel November 5, 2023 17:51
@CodyCBakerPhD
Copy link

Is there a way to standardize the position of the badge display on the docs?

image

@oruebel
Copy link
Contributor

oruebel commented Nov 5, 2023

Is there a way to standardize the position of the badge display on the docs?

It's a matter of the size of the logo. I.e., eventually the text wraps around the logo. If you want everything to show on the right of the logo, you'd need to add empty space to the logo.

@bendichter
Copy link
Collaborator Author

bendichter commented Nov 5, 2023

or we could create a more sophisticated layout, like creating tables or divs, but I'm not exactly sure how well that would place with our current approach of referencing text from within other files. I think this page might eventially benefit from some templating, e.g. hugo.

@bendichter bendichter requested a review from rly December 5, 2023 13:52
@bendichter
Copy link
Collaborator Author

HDF5 links are down temporarily:

image

@bendichter bendichter merged commit a6f240b into main Dec 5, 2023
2 of 3 checks passed
@bendichter bendichter deleted the add_github_star_badges branch December 5, 2023 17:27
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