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

resolves #3893 allow footnotes in footnotes #3894

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hedrok
Copy link
Contributor

@hedrok hedrok commented Dec 28, 2020

I've tryed my best at following CONTRIBUTING.adoc, but I have no experience with Ruby and ascidoctor code.

Details:

  • Add 2 tests: basic support of footnotes in footnotes
    and support for escaped ]
  • Change InlineFootnoteMacroRx so that it allows recursive footnotes
  • Move footnote processing code from sub_macros to separate function
    sub_footnote, call it recursively on matches. It is important not
    to call sub_macros recursively as this results to multiple expansion
    of some macros (e.g. URL)

* Add 2 tests: basic support of footnotes in footnotes
  and support for escaped ]
* Change InlineFootnoteMacroRx so that it allows recursive footnotes
* Move footnote processing code from sub_macros to separate function
  sub_footnote, call it recursively on matches. It is important not
  to call sub_macros recursively as this results to multiple expansion
  of some macros (e.g. URL)
@mojavelinux mojavelinux deleted the branch asciidoctor:main October 23, 2021 07:57
@mojavelinux mojavelinux reopened this Oct 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants