From: Thomas Lamprecht <t.lamprecht@proxmox.com>
To: marcandre.lureau@redhat.com, qemu-devel@nongnu.org
Cc: berrange@redhat.com, jsnow@redhat.com, peter.maydell@linaro.org
Subject: Re: [PATCH v5] sphinx: adopt kernel readthedoc theme
Date: Tue, 23 Mar 2021 13:25:34 +0100 [thread overview]
Message-ID: <4144cbea-4aa9-359c-f01e-34c16f2c1efe@proxmox.com> (raw)
In-Reply-To: <20210323115328.4146052-1-marcandre.lureau@redhat.com>
On 23.03.21 12:53, marcandre.lureau@redhat.com wrote:
> From: Marc-André Lureau <marcandre.lureau@redhat.com>
>
Just saw this patch by accident and as we also use the alabaster theme
for the Proxmox Backup project I wanted to share some insights from our
usage, as I checked that theme out closely a few months ago and did some
adaptions for, partially overlapping, short-comings we found.
> The default "alabaster" sphinx theme has a couple shortcomings:
> - the navbar moves along the page
That can be fixed with the following conf.py 'html_theme_options' setting:
'fixed_sidebar': True,
https://git.proxmox.com/?p=proxmox-backup.git;a=blob;f=docs/conf.py;h=cfa4158d6b284172929785991f710d6237e9992c;hb=2ab2ca9c241f8315f51f9c74a50d7223c875a04b#l161
> - the search bar is not always at the same place
Can be also addressed by setting 'html_sidebars' to a fixed order, e.g.:
html_sidebars = {
'**': [
'searchbox.html',
'navigation.html',
'relations.html',
]
}
Can also be customized for different pages, e.g., we do so for landing pages:
https://git.proxmox.com/?p=proxmox-backup.git;a=blob;f=docs/conf.py;h=cfa4158d6b284172929785991f710d6237e9992c;hb=2ab2ca9c241f8315f51f9c74a50d7223c875a04b#l188
I added also a short JS snipped to scroll the heading of the current chapter in
the sidebar TOC into view (adapted from rust book).
https://git.proxmox.com/?p=proxmox-backup.git;a=blob;f=docs/custom.js;h=7964b2cb0ea9433596845618f1679f1672ce38b8;hb=2ab2ca9c241f8315f51f9c74a50d7223c875a04b
If you want, you could check out the result at our hosted docs site:
https://pbs.proxmox.com/docs/managing-remotes.html
> - it lacks some contrast and colours
That is true, and IMO the rtd theme really uses a better colour palette,
especially for things like "Topic" blocks.
In fact we pondered switching over to rtd, so please don't see my mail
as me advertising that all issues can be fixed into alabaster, just wanted
to share what we did to overcome the first two short-comings mentioned here.
cheers,
Thomas
next prev parent reply other threads:[~2021-03-23 12:28 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-23 11:53 [PATCH v5] sphinx: adopt kernel readthedoc theme marcandre.lureau
2021-03-23 12:25 ` Thomas Lamprecht [this message]
2021-03-23 12:34 ` Marc-André Lureau
2021-03-25 20:05 ` Marc-André Lureau
2021-03-25 20:10 ` Marc-André Lureau
2021-03-26 19:28 ` John Snow
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=4144cbea-4aa9-359c-f01e-34c16f2c1efe@proxmox.com \
--to=t.lamprecht@proxmox.com \
--cc=berrange@redhat.com \
--cc=jsnow@redhat.com \
--cc=marcandre.lureau@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
/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;
as well as URLs for NNTP newsgroup(s).