Process HTML <img> tags in addition to markdown image markup when embedding local images

[earshinov]

Extends #282 and #460.

Demo (copied from extended testdata/demo_long.py):

<img src="../../docs/logo.png" alt="pdoc logo" width="150">

I am interested in this because there is no "official" way to specify image width and height in Markdown markup. markdown2 doesn't seem to recognize the ![pdoc logo](../../docs/logo.png =150x) syntax supported by some other Markdown implementations.

[earshinov]

Also, not sure why autofix.ci introduced so many changes on top of my commit

[mhils]

Thanks for the PR! This looks good. Please 1) rebase onto main and 2) add a CHANGELOG entry.

The autofix.ci changes were because you updated uv.lock, which updated our dependencies, which updated the HTML generated by pdoc for snapshot tests. I've done the same in #788, so this PR should be clean now.

[earshinov]

Thanks for the PR! This looks good. Please 1) rebase onto main and 2) add a CHANGELOG entry.

Done

[mhils]

Can you please keep the whitespace changes in CHANGELOG.md at bay? I don't mind the occasional fly-by change, but (some) whitespace in Markdown has semantic meaning.

[earshinov]

Can you please keep the whitespace changes in CHANGELOG.md at bay? I don't mind the occasional fly-by change, but (some) whitespace in Markdown has semantic meaning.

Sorry, fixed now.

By the way, have you considered using explicit <br>'s instead of trailing whitespace to force line breaks? This way people won't break the layout inadvertently. I know I'm not the first one 🙂

[mhils]

By the way, have you considered using explicit <br>'s instead of trailing whitespace to force line breaks? This way people won't break the layout inadvertently. I know I'm not the first one 🙂

Subjective preference for Markdown over HTML. I don't feel super strongly, but it's a Markdown document so using Markdown syntax is fine IMO. :)

Thanks again for the PR!