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

"No link definition found" for footnotes #8

Open
dtinth opened this issue Jul 24, 2022 · 3 comments
Open

"No link definition found" for footnotes #8

dtinth opened this issue Jul 24, 2022 · 3 comments
Labels
enhancement New feature or request
Milestone

Comments

@dtinth
Copy link

dtinth commented Jul 24, 2022

When I use footnotes1, the language server generates a warning.

image

Here’s a repro, put this is src/test/diagnostic.test.ts:

	test('Should not generate diagnostic for valid footnotes', withStore(async (store) => {
		const doc = new InMemoryDocument(workspacePath('doc.test.md'), joinLines(
			`footnotes should work[^fn1]`,
			``,
			`[^fn1]: this is a footnote`,
		));

		const workspace = store.add(new InMemoryWorkspace([doc]));

		const diagnostics = await getComputedDiagnostics(store, doc, workspace);
		assertDiagnosticsEqual(diagnostics, []);
	}));

Footnotes

  1. https://github.blog/changelog/2021-09-30-footnotes-now-supported-in-markdown-fields/

@mjbvz
Copy link
Contributor

mjbvz commented Jul 25, 2022

Footnotes aren't part of CommonMark so this is expected

They are common enough that I'd be open to adding support for them, but that should be something the library consumer has to opt into

@mjbvz mjbvz added the enhancement New feature or request label Sep 22, 2022
@hellt
Copy link

hellt commented Oct 14, 2022

I second that, footnotes are awesome and it'd be great for vscode markdown services to parse them accordingly

@mail2nnv

This comment was marked as spam.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants