Skip to content

Conversation

@wesleyboar
Copy link
Collaborator

@wesleyboar wesleyboar commented Oct 7, 2025

Peer asked me about doing this. I'll show the result on pre-prod for stakeholder meeting tomorrow, where I offer his request be presented.

Important

Hazmapper:

pre-prod Hazmapper

pre-prod.deep.mov

prod Hazmapper

prod.shallow.mov

Peer asked me about doing this. I'll show the result on pre-prod for stakeholder meeting tomorrow, where I offer his request be presented.
@silviamazzoni
Copy link
Collaborator

I'm always a fan of freedom and opportunity.... when you go so deep you either have to make the left nav much wider, of you have to use short words. but that's up to the author.

@wesleyboar
Copy link
Collaborator Author

@silviamazzoni, yup.

I worry this affects more than just the document of concern (unknown, awaiting answer from peer). I'll check tomorrow before meeting (as time allows) which nav items will newly show up, and which are just too long for this.

@taoteg
Copy link
Collaborator

taoteg commented Oct 8, 2025

I think the level 6 depth would work okay as is.
I don't see any unintended changes to the nav design by this change.
The only places in the existing docs that this unintentionally extends are these:

  • Hazard Apps > VORTEX-Winds: DEDM-HR > Jupyter Notebook for DEDM-HR > Implementation > entries_visible
  • Several Use Cases would also reveal new L5 headings (a few have L6 - Seismic section).

The nested headings at L6 aren't gorgeous but they are usable (imho).
Is there a way we could style the L6 links to not be so heavy and large?
Maybe a plain or italicized text for the L6 links at a smaller font?
Or possibly just all the leaf nodes get that treatment?
Dunno if that is possible with the mkdocs theme setup.

But the main reason for adopting imo is that all the linking just works out of the box using the markdown without having to resort to html hacks or weird inconsistencies in the config setup.

@wesleyboar
Copy link
Collaborator Author

Thanks for checking what else is affected. I had no bandwidth yesterday. Yes, I can make deeper headings compete less with existing headings. I anticipate no problem with this change.

@wesleyboar wesleyboar marked this pull request as ready for review October 8, 2025 16:25
@wesleyboar wesleyboar marked this pull request as draft October 8, 2025 16:25
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