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

Adjust initial_header_level for TOC #1080

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sjbronner
Copy link

The initial_header_level, set to 2, causes 2 problems:

  1. When including a TOC, it will have an empty first level, which just looks really bad.
  2. It differs from the way headings are handled in MarkDown.

The initial_header_level, set to 2, causes 2 problems:

1. When including a TOC, it will have an empty first level, which just looks really bad.
2. It differs from the way headings are handled in MarkDown.
@smokenasty4u
Copy link

What's the issue

@sjbronner
Copy link
Author

The issue is not apparent on github because the generation of a table of contents is not available for restructuredText there. The issue appeared in my company's blog on pages written in restructuredText and looks something like this (table of contents):

    • First Section Title
    • Second Section Title
      • Subsection Title
      • Another Subsection Title
    • Third Section Title

It should look like this, and this is what results from my pull request:

  • First Section Title
  • Second Section Title
    • Subsection Title
    • Another Subsection Title
  • Third Section Title

Copy link

@panderson02 panderson02 left a comment

Choose a reason for hiding this comment

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

How to change my icon on IPhone 7+ Please help

@3572
Copy link

3572 commented Feb 22, 2018

NToyKala

Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the Stale label Dec 10, 2024
@10728811
Copy link

10728811 commented Dec 10, 2024 via email

@sjbronner
Copy link
Author

@panderson02 You reviewed this issue in February, but it is still marked as "Review required". Is something missing?

@github-actions github-actions bot removed the Stale label Dec 11, 2024
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.

6 participants