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

second level nav missing #1673

Merged
merged 3 commits into from
Jul 18, 2023

Conversation

BrewCityBoy
Copy link
Contributor

@BrewCityBoy BrewCityBoy commented Jul 17, 2023

Fix for missing indentation of "Pages" links in Design System sidenav. Taken from Issue 1294:

It's weird that https://cfpb.github.io/design-system/pages/ doesn't have indented links, unlike every other section:

@BrewCityBoy BrewCityBoy self-assigned this Jul 17, 2023
@BrewCityBoy BrewCityBoy linked an issue Jul 17, 2023 that may be closed by this pull request
@netlify
Copy link

netlify bot commented Jul 17, 2023

Thanks for the improvements! Browse a preview of your changes using the link below.

Name Link
🔨 Latest commit e2a893d
🔍 Latest deploy log https://app.netlify.com/sites/cfpb-design-system/deploys/64b69b221b94f000089b6d21
😎 Deploy Preview https://deploy-preview-1673--cfpb-design-system.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@anselmbradford
Copy link
Member

Hmm if I go to https://deploy-preview-1673--cfpb-design-system.netlify.app/design-system/pages/ now there are NO links!

@BrewCityBoy
Copy link
Contributor Author

Hmm if I go to https://deploy-preview-1673--cfpb-design-system.netlify.app/design-system/pages/ now there are NO links!

Perhaps the script in sidebar.html is looking for the h5 'eyebrow' tag, but none of the pages md files have that tag? I think it's the file structure.

@BrewCityBoy
Copy link
Contributor Author

Scratch the idea about the file structure in the 'md' files. The "pages" needed to be indented into the second level in the side-navigation.yml file.

@anselmbradford
Copy link
Member

Fixes #1294

@BrewCityBoy BrewCityBoy merged commit 01112ea into main Jul 18, 2023
@BrewCityBoy BrewCityBoy deleted the josh_design-system_pages_sidenav_indentation branch July 18, 2023 17:54
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.

Design System docs pages page doesn't have indented links
2 participants