bug: deprecation warning with latest mkdocstrings and autorefs

[mschoettle]

Description of the bug

The following deprecation warning gets output with the latest mkdocstrings version:

https://github.com/mkdocstrings/autorefs/blob/40940c591e9cc557573ccd35351ca2b535ec371b/src/mkdocs_autorefs/references.py#L274-L279

I am aware that this is the legacy Python handler. Despite that, is there still interest in having this fixed?

[pawamoy]

Yes, this should be fixed! Thanks for reporting.

[mschoettle]

Great!

I tried replacing this here:

python-legacy/src/mkdocstrings_handlers/python/rendering.py

Line 28 in c8aacd7

return Markup("<span data-autorefs-optional-hover={path}>{brief}</span>").format(path=path, brief=brief)

with:

return Markup("<autorefs optional-hover={path}>{brief}</autorefs>").format(path=path, brief=brief)

It fixes the warning, not sure if it's correct though. A quick comparison of some rendered output seems to be identical.

If that looks good I can create a PR.

[pawamoy]

It should be this:

return Markup("<autoref identifier={path} optional hover>{brief}</autoref>").format(path=path, brief=brief)

PR welcome! I should be able to review+merge+release tomorrow :)