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

Fix an Infinite Loop for re-index-footnotes and move-footnote-to-the-bottom #785

Merged
merged 1 commit into from
Jul 9, 2023

Conversation

pjkaufman
Copy link
Collaborator

Fixes #784

Changes Made:

  • Accounted for starting a file with a footnote reference in the while loops which was caused by the condition being greater than -1 when it should be 0 since that is the lowest valid value that we should stop looking after we find
  • Added a test to move-footnote-to-the-bottom to make sure the issue does not get reintroduced

@pjkaufman pjkaufman added bug Something isn't working footnote Deals with markdown footnotes labels Jul 9, 2023
@pjkaufman pjkaufman self-assigned this Jul 9, 2023
@pjkaufman pjkaufman merged commit c5dae1f into platers:master Jul 9, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working footnote Deals with markdown footnotes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: re-index-footnotes & move-footnotes-to-the-bottom can cause Obsidian freeze with footnote wrapping
1 participant