-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Add aside.topic
for Docutils 0.18+
#10493
Conversation
Is the rendered HTML broken if we don't merge this? I'd like to know how important this fix is. |
Yes, one second for a screenshot A |
Script:import shutil
import types
from pathlib import Path
from docutils.parsers.rst.states import Body
from docutils.statemachine import StringList
from sphinx.cmd.make_mode import run_make_mode
from sphinx.ext.autodoc.directive import parse_generated_content
def write(filename, text): Path(filename).write_text(text, encoding="utf-8")
write("conf.py", '''\
html_theme = "classic"
''')
write("index.rst", '''\
***************************************
An introduction to the ipaddress module
***************************************
.. topic:: Overview
This document aims to provide a gentle introduction to the
:mod:`ipaddress` module. It is aimed primarily at users that aren't
already familiar with IP networking terminology, but may also be useful
to network engineers wanting an overview of how :mod:`ipaddress`
represents IP network addressing concepts.
''')
shutil.rmtree("_build", ignore_errors=True)
run_make_mode(["html", ".", "_build", "-T", "-W"]) A |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. It must be an important bug. So I'm merging this into the 5.0.x branch.
This fix is still missing for Sphinx's own theme
I think this is a good example for how it is easy to miss fixes in the |
I wouldn't say it's your fault, because the |
Found when rebuilding the Python documentation with Sphinx 5.
I'm not sure if this is worth a
5.0.1
release or not.Feature or Bugfix
A