[sphinx_roles] abbr eplanation not taken into account

[matclab]

Using :abbr:`WTF (Want To Fly)` get translated into <abbr>WTF</abbr> with sphinx_roles plugin 0.4 on Nikola 8.0.1 with python 3.7.1 on Arch Linux;

Putting some debugging info, it looks like visit_abbreviation is never called.

[ralsina]

Hmmm ... ok, taking a look.

[ralsina]

Ok, seems like I should actually remove this soonish.

According to sphinx source code, this is deprecated because docutils has added abbreviation support.
And according to the docutils code, it's not working:

    def visit_abbreviation(self, node):
        # @@@ implementation incomplete ("title" attribute)

I will take a shot at fixing it but I have no idea why the visitor is not working.

[matclab]

@ralsina, by chance did you manage to get some result on this one ?

[matclab]

after some investigation, it looks like add_node change the visitor of docutils.writers.html5_polyglot.HTMLTranslator whereas the abbr_visitor used is the one from docutils.writers.html4css1.HTMLTranslator.

[matclab]

Replacing html5_polyglot with html4css1 in nikola/plugins/compile/rest/__init__.py works. Shall I propose a PR or could you elaborate on the difference between the two writers and why one is selected preferably to another ?

[ralsina]

Honestly, I have no idea what the difference is :-(

…

On Mon, Jul 29, 2019 at 4:48 PM matclab ***@***.***> wrote: Replacing html5_polyglot with html4css1 in nikola/plugins/compile/rest/__init__.py works. Shall I propose a PR or could you elaborate on the difference between the two writers and why one is selected preferably to another ? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#294?email_source=notifications&email_token=AAAAMK4Y2F7W3FFWJTH2KBDQB5CPBA5CNFSM4GOMUBWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3BZ2DQ#issuecomment-516136206>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAAMKZCG22VPSHPD6FODNDQB5CPBANCNFSM4GOMUBWA> .

[matclab]

The difference is documented here : http://docutils.sourceforge.net/docs/user/html.html
Replacing writer_name='html' with writer_name='html5_polyglot' in nikola/plugins/compile/rest/__init__.py:rst2html parameter initialization does the job.

[matclab]

But then the math rendering is broken… Still investigating.

[matclab]

The math problem is unrelated… It was a glitch in the network connection.

[ralsina]

I suppose if the tests pass it's ok.

…

On Mon, Jul 29, 2019 at 5:08 PM matclab ***@***.***> wrote: The difference is documented here : http://docutils.sourceforge.net/docs/user/html.html Replacing writer_name='html' with writer_name='html5_polyglot' in nikola/plugins/compile/rest/__init__.py:rst2html parameter initialization does the job. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#294?email_source=notifications&email_token=AAAAMK6UGCD4KOCQ5LNKKLDQB5E3RA5CNFSM4GOMUBWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3B3NQA#issuecomment-516142784>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAAMK2Y2TZS5YJ5IGK6WWLQB5E3RANCNFSM4GOMUBWA> .