Linux Documentation
 help / color / mirror / Atom feed
From: Jonathan Corbet <corbet@lwn.net>
To: Antonin Godard <antonin.godard@bootlin.com>,
	Shuah Khan <skhan@linuxfoundation.org>
Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
	Antonin Godard <antonin.godard@bootlin.com>,
	Randy Dunlap <rdunlap@infradead.org>
Subject: Re: [PATCH v2 0/2] Documentation: html: show sections in the sidebar
Date: Wed, 05 Aug 2026 13:41:16 -0600	[thread overview]
Message-ID: <87tsp8nyn7.fsf@trenco.lwn.net> (raw)
In-Reply-To: <20260804-show-sections-in-sidebar-v2-0-a68d23bbc79d@bootlin.com>

Antonin Godard <antonin.godard@bootlin.com> writes:

> The current sidebar in the HTML version of the documentation does not
> display the section titles because the toctree directives in the
> top-level index.rst document do not contain ":caption:" properties.
> Replacing the current section titles by ":caption:" properties would not
> allow having text between those and the table of contents.
>
> To workaround this issue, add the ":caption:" properties in the toctree
> calls which makes them show up in the sidebar, but hide them from the
> index page with a custom CSS addition.
>
> Additionally, make the section titles in the sidebar bold to make them
> stand-out.
>
> This makes the overall structure of the documentation clearer from the
> sidebar directly.
>
> PS: This is how I've implemented this in the Yocto Project
> documentation[1] where I faced the same issue. See also the index.rst
> file[2] (which was by the way inspired by the kernel's own index.rst)
> and CSS addition[3].
>
> [1]: https://docs.yoctoproject.org/dev/
> [2]: https://git.yoctoproject.org/yocto-docs/tree/documentation/index.rst
> [3]: https://git.yoctoproject.org/yocto-docs/tree/documentation/sphinx-static/theme_overrides.css#n106
>
> Signed-off-by: Antonin Godard <antonin.godard@bootlin.com>
> ---
> Changes in v2:
> - Apply RB and TB from Randy on patch 1/2.
> - Remove whitespace before section title and first subsection.
> - Link to v1: https://patch.msgid.link/20260803-show-sections-in-sidebar-v1-0-b4fdc847525d@bootlin.com

You didn't answer my question about the PDF build.  It's easy to break
that build, and easy to forget to check it...trust me, I know.  I did
check it, and it all seems fine.

So the patches are applied, thanks.  I did tweak the CSS slightly:

diff --git a/Documentation/sphinx-static/custom.css b/Documentation/sphinx-static/custom.css
index 34aaa424a75c..6c03dca44c82 100644
--- a/Documentation/sphinx-static/custom.css
+++ b/Documentation/sphinx-static/custom.css
@@ -87,7 +87,8 @@ section#the-linux-kernel-documentation p.caption { display: none; }
  * Make section titles bold in the sidebar, and decrease their bottom margin to
  * group them with their subsections.
  */
-div.sphinxsidebar p.caption { font-weight: bold; margin-bottom: -10px; }
+div.sphinxsidebar p.caption { font-weight: bold; margin-bottom: 0; }
+div.sphinxsidebar p.caption + ul { margin-top: 0; }
 
 /*
  * The CSS magic to toggle the contents on small screens.

This way more directly expresses the intent, and will (hopefully) cause
things to continue to work properly in the face of an Alabaster version
that changes the margins.

Thanks,

jon

  parent reply	other threads:[~2026-08-05 19:41 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-04 15:59 [PATCH v2 0/2] Documentation: html: show sections in the sidebar Antonin Godard
2026-08-04 15:59 ` [PATCH v2 1/2] " Antonin Godard
2026-08-04 15:59 ` [PATCH v2 2/2] Documentation: html: adjust sidebar section titles styling Antonin Godard
2026-08-04 16:49   ` Randy Dunlap
2026-08-05 19:41 ` Jonathan Corbet [this message]
2026-08-06  8:57   ` [PATCH v2 0/2] Documentation: html: show sections in the sidebar Antonin Godard
2026-08-06 13:10     ` Jonathan Corbet

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87tsp8nyn7.fsf@trenco.lwn.net \
    --to=corbet@lwn.net \
    --cc=antonin.godard@bootlin.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rdunlap@infradead.org \
    --cc=skhan@linuxfoundation.org \
    --cc=thomas.petazzoni@bootlin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox