image not respecting alignement with png in .rst

[12rambau]

I decided to open a follow-up of this thread as the bug seems to be fixed but fails on some more specific image type.

I try to align the content of my readme.rst file to use as a long description in my pypi release. using the .. raw:: directive is then impossible.

A common way to do it (that works on both sphinx and pypi) is:

.. image:: toto.png
   :align: center

It seems this is not taken into account when generating .png preview as shown in this gist where the .svg is correctly aligned and the .png is not.

[jluebbe]

It seems that alignment also doesn't work for URLs with parameters (?foo=1) after the .svg.

[12rambau]

Actually this is not respected by .sgv image anymore either. I think it's related to all the regression issue reported since Jan #1798

[github-actions]

Stale issue message

[lafrech]

This thread might be stale, but the issue is still there.

[flying-sheep]

@zkoppert please reopen

[zkoppert]

Reopening this issue based on investigation in #1222. The bug is confirmed and reproducible — RST image alignment works for SVGs but not for PNGs (or any non-SVG image type).

Root cause: Our GitHubHTMLTranslator in rest2html has custom depart_image handling for SVGs that rebuilds the <img> tag and copies the align attribute directly. Non-SVG images fall through to docutils' default HTMLTranslator, which converts :align: to class="align-right" — and the sanitization pipeline strips class attributes. The same issue affects :width: and :height:, which get converted to style attributes that are also stripped.

Proposed fix: Update GitHubHTMLTranslator to emit align, width, and height as HTML attributes for all images (not just SVGs). These attributes are explicitly in the sanitizer's allowlist and are safe — they can only express layout, not execute code.