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

feat: allow multi-line footnotes #14

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

Conversation

heckman
Copy link

@heckman heckman commented Nov 24, 2023

Hello,

Please bear with me as this is the first pull request I have made.

The only thing of consequence that I changed in your code
is the regex on line 4 of src/utils/index.ts.

The other changes were just me attempting to create a new extension.
(I've never made a VS Code extension either.)

—eh

commit message follows


The existing footnote-body-matching regex
requires a space after the label's colon;
i.e., the line needs to start like '[^label]: '.

This does not provide for multiline footnote bodies,
where the first line can end with the ':'
and is followed by one or more indented lines.

The regex pattern introduced in this commit
does not require the existence of subsequent lines,
nor that any subsequent lines be indented.

Date: Fri Nov 24 15:11:48 2023 -0500

On branch heckman
Changes to be committed:
modified: CHANGELOG.md
modified: README.md
modified: package-lock.json
modified: package.json
modified: src/utils/index.ts
modified: test/workspace/test.md

The existing footnote-body-matching regex
requires a space after the label's colon;
i.e., the line needs to start like '[^label]: '.

This does not provide for multiline footnote bodies,
where the first line can end with the ':'
and is followed by one or more indented lines.

The regex pattern introduced in this commit
does not require the existence of subsequent lines,
nor that any subsequent lines be indented.

Date:      Fri Nov 24 15:11:48 2023 -0500

On branch heckman
Changes to be committed:
	modified:   CHANGELOG.md
	modified:   README.md
	modified:   package-lock.json
	modified:   package.json
	modified:   src/utils/index.ts
	modified:   test/workspace/test.md
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.

1 participant