Skip to content

Conversation

@GBKS
Copy link
Contributor

@GBKS GBKS commented Jun 24, 2025

Introduced a table of contents section at the top of the 'Units and symbols' guide to improve navigation and usability.

🧜‍♀️Check the preview🧞

Introduced a table of contents section at the top of the 'Units and symbols' guide to improve navigation and usability.
@GBKS GBKS self-assigned this Jun 24, 2025
@netlify
Copy link

netlify bot commented Jun 24, 2025

Deploy Preview for bitcoin-design-site ready!

Name Link
🔨 Latest commit 5851940
🔍 Latest deploy log https://app.netlify.com/projects/bitcoin-design-site/deploys/685a430df1898300081bb517
😎 Deploy Preview https://deploy-preview-1178--bitcoin-design-site.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 project configuration.

@GBKS GBKS requested a review from Copilot June 24, 2025 06:20
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a manual table of contents at the top of the “Units and symbols” guide to improve navigation and usability.

  • Introduces a no_toc class to suppress auto-generated TOC
  • Inserts custom TOC block wrapped in a .glossary-toc container
  • Adds horizontal rules to visually separate the TOC from the main content
Comments suppressed due to low confidence (2)

guide/designing-products/units-and-symbols.md:80

  • [nitpick] It may help future maintainers to include a brief HTML comment explaining the purpose of .glossary-toc here or link to the style definition, so the custom container usage is clear.
<div class="glossary-toc" markdown="1">

guide/designing-products/units-and-symbols.md:78

  • [nitpick] Using --- for horizontal rules can conflict with frontmatter parsing in some Markdown engines. Consider switching to *** or ensuring this delimiter is supported by the site build pipeline.
---

Copy link
Collaborator

@swedishfrenchpress swedishfrenchpress left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@swedishfrenchpress swedishfrenchpress merged commit 8aecad5 into master Jun 24, 2025
4 checks passed
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