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

ID ordering must do depth first not breadth first #4

Closed
cirosantilli opened this issue Mar 23, 2020 · 0 comments
Closed

ID ordering must do depth first not breadth first #4

cirosantilli opened this issue Mar 23, 2020 · 0 comments
Milestone

Comments

@cirosantilli
Copy link
Contributor

cirosantilli commented Mar 23, 2020

At cirodown 3254f4a , we can get surprising ID orders e.g.:

\M[1]

\q[\M[2]]

\M[3]

produces IDs that render on the document in order 1 3 2 rather than the desired 1 2 3.

This is because the \q is only searched for IDs after the toplevel \M[3].

Generally this is not a big problem because linkable things stay on toplevel, but still, should be fixed.

Maybe this will also completely break math macro definitions across math blocks: https://github.com/cirosantilli/cirodown/blob/73b52c4ce0e50945724a843ac12212faa2ded4bf/README.ciro#L870

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

No branches or pull requests

1 participant