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

toc.integrate causes undesired indentation in left nav #6779

Closed
4 tasks done
castedo opened this issue Feb 12, 2024 · 2 comments
Closed
4 tasks done

toc.integrate causes undesired indentation in left nav #6779

castedo opened this issue Feb 12, 2024 · 2 comments
Labels
bug Issue reports a bug resolved Issue is resolved, yet unreleased if open

Comments

@castedo
Copy link

castedo commented Feb 12, 2024

Context

Material for MkDocs is awesome! I love it!

Here is a fix to this bug:
https://github.com/castedo/mkdocs-material-bug1/blob/main/docs/style/extra.css

Probably not the right fix, but it is what I current hack into my websites.

Bug description

When I build with mkdocs-material 9.4.8 and 9.4.14 and 9.5.9 and use theme: features: toc.integrate I see an unexpected indentation in the left nav.

With the attached repo case, I see the following:
dislike_nav

But what I expected and wanted is:
nav_expected
which is consistent with what I see in the docs:
setup docs

Related links

Reproduction

repro.zip

You can also clone https://github.com/castedo/mkdocs-material-bug1 which has the same contents as the zip.

Steps to reproduce

mkdocs build or mkdocs serve on the repro example above.

You can uncomment the extra_css: - styles/extra.css lines to fix the problem.

Browser

No response

Before submitting

@squidfunk squidfunk added the bug Issue reports a bug label Feb 13, 2024
@squidfunk
Copy link
Owner

Thanks for reporting. It appears that this was caused by remnants of the latest navigation refactoring. 7bbc9f9 fixes the reported issue, and another problem I discovered where the integrated table of contents of top-level pages when navigation tabs are activated is moved to the left.

@squidfunk squidfunk added the resolved Issue is resolved, yet unreleased if open label Feb 19, 2024
@squidfunk
Copy link
Owner

Released as part of 9.5.10.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue reports a bug resolved Issue is resolved, yet unreleased if open
Projects
None yet
Development

No branches or pull requests

2 participants