Skip to content
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

Log warning if footnoteref macro is found (unless compat mode is enabled) #3114

Closed
mojavelinux opened this issue Mar 1, 2019 · 0 comments · Fixed by #3115
Closed

Log warning if footnoteref macro is found (unless compat mode is enabled) #3114

mojavelinux opened this issue Mar 1, 2019 · 0 comments · Fixed by #3115
Assignees
Milestone

Comments

@mojavelinux
Copy link
Member

The footnoteref macro has been replaced by the footnote macro with a target. This change was introduced in 1.5.7. Starting in 2.0.0, the processor should log a warning if the footnoteref macro is found, unless compat mode is enabled.

The warning should be fashioned after this example:

found deprecated footnoterefmacro: footnoteref:[refid,footnote text]; use footnote macro with target instead

The preferred syntax is footnote:idname[footnote text]. The reason for the warning is that the footnoteref macro doesn't follow the pattern of all the other macros and therefore propagates a misconception. For example, I've seen extensions model the syntax of a custom macro after footnoteref and I really don't want to discourage that. Consistency is key.

@mojavelinux mojavelinux added this to the v2.0.0 milestone Mar 1, 2019
@mojavelinux mojavelinux self-assigned this Mar 1, 2019
mojavelinux added a commit to mojavelinux/asciidoctor that referenced this issue Mar 1, 2019
mojavelinux added a commit to mojavelinux/asciidoctor that referenced this issue Mar 1, 2019
mojavelinux added a commit that referenced this issue Mar 1, 2019
commonquail added a commit to commonquail/git-mr-to that referenced this issue Apr 5, 2021
Asciidoctor has preferred another way to write this style of footnotes
for years [1]. The general pattern of the change is

    -footnoteref:[myid,my text].
    +footnote:myid[my text].

[1] asciidoctor/asciidoctor#3114
commonquail added a commit to commonquail/commitmsgfmt that referenced this issue Apr 5, 2021
Asciidoctor has preferred another way to write this style of footnotes
for years [1]. The general pattern of the change is

    -footnoteref:[myid,my text].
    +footnote:myid[my text].

However, unlike the footnoteref macro prints "my text" verbatim whereas
the footnote macro processes it like extra text, so the URL used here
has to be escaped to disable autolinking.

[1] asciidoctor/asciidoctor#3114
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant