Group remote-dataset cards on the index page by category - #152
Merged
Conversation
The landing page listed all 22 datasets in one flat grid, in a historical/ad-hoc order that mixed relief, gravity, magnetics, and imagery together. Split it into labeled sections (Miscellaneous, Gravity, Geodesy, Magnetics, Topography, Images, Other Celestial Bodies) that mirror the categories already used by the gmtserver-admin Makefile, and split the single hidden toctree into one per section so the same groups show up in the left sidebar navigation.
Member
|
I thinks this will be good but I would like to see how it looks. |
Member
Author
|
Member
Author
|
I should probably use alphabetical order. |
Geodesy, Gravity, Images, Magnetics, Miscellaneous, Other Celestial Bodies, Topography - both the visible grid sections and the corresponding hidden toctree captions in the sidebar.
Member
Author
|
Now in alphabetical order. |
|
|
||
| .. toctree:: | ||
| :hidden: | ||
| :caption: Topography |
Member
There was a problem hiding this comment.
I like the groupping of the datasets, but I feel it's not optimal to have the Earth relief grids as the last entry. By renaming the heading to "Elevation and bathymetry", it could be moved to the top following the alphabetical order.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The landing page (
docs/index.rst) listed all 22 remote datasets in a single flat grid. This PR groups the cards into labeled sections (that mirror the categories already used in thegmtserver-admin):The single hidden
toctreewas also split into onetoctreeper section (each with its own:caption:), so the same groups show up as separate blocks in the left sidebar navigation instead of one long undifferentiated list.