Skip to content

Make index page match toctree structure #4

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

Conversation

sneakers-the-rat
Copy link

@sneakers-the-rat sneakers-the-rat commented Feb 27, 2024

Part of: pyOpenSci#181

As always, with any of my comments or reviews, take this or leave it! i won't be offended if you don't like these changes :).

Currently it's not obvious that the content on the index page is related to the tabs at the top of the page. This makes the overall structure of the site a bit bewildering - I'm not quite sure if this index is unrelated to the tabbed sections, and it's difficult to scan through the boxes in the grid quickly.

This PR reorganizes the index page to match the toctree, adding titles and links to other subpages. It might be a few too many links without text, we can take out some of the docs links, but hopefully this makes the different groups a bit clearer.

I changed the order of the toctree to put packaging before documentation because the blocks on the index for packaging came first, but that can also go back, i just think they should match the order!

before:

Screenshot 2024-02-27 at 1 17 15 PM

after:

Screenshot 2024-02-27 at 1 16 20 PM

@lwasser lwasser force-pushed the license-pyproject branch from cdfd0c6 to ae4a4aa Compare March 1, 2024 17:41
@lwasser
Copy link
Owner

lwasser commented Mar 1, 2024

once i merge the open PR - let's rebase this against main and then look at the build. i think that the structure IS confusing now and this will make it better. let's just add it as a separate pr if you are ok with that! thank you so so much for this @sneakers-the-rat

@sneakers-the-rat
Copy link
Author

sounds gud :)

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