diff options
author | Matt Bierner <matb@microsoft.com> | 2022-06-01 22:01:36 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-06-01 22:01:36 +0300 |
commit | 7477efd87ecf9bffea9ad2bfc5d3c3657380127f (patch) | |
tree | af4a2ffb52927b5c020c878c579e6e085628ce51 /extensions | |
parent | 2664c7ddfd7e14455f39a89360f31d63d56ffc3e (diff) |
Better message for markdown reference links (#151018)
Fixes #151017
Also improves the diagnostic message for invalid reference links to make it clear we are talking about a missing link definition
Diffstat (limited to 'extensions')
-rw-r--r-- | extensions/markdown-language-features/src/languageFeatures/diagnostics.ts | 2 | ||||
-rw-r--r-- | extensions/markdown-language-features/src/languageFeatures/documentLinkProvider.ts | 4 |
2 files changed, 4 insertions, 2 deletions
diff --git a/extensions/markdown-language-features/src/languageFeatures/diagnostics.ts b/extensions/markdown-language-features/src/languageFeatures/diagnostics.ts index 10547f2000e..4db32d614f7 100644 --- a/extensions/markdown-language-features/src/languageFeatures/diagnostics.ts +++ b/extensions/markdown-language-features/src/languageFeatures/diagnostics.ts @@ -448,7 +448,7 @@ export class DiagnosticComputer { if (link.href.kind === 'reference' && !definitionSet.lookup(link.href.ref)) { yield new vscode.Diagnostic( link.source.hrefRange, - localize('invalidReferenceLink', 'No link reference found: \'{0}\'', link.href.ref), + localize('invalidReferenceLink', 'No link definition found: \'{0}\'', link.href.ref), severity); } } diff --git a/extensions/markdown-language-features/src/languageFeatures/documentLinkProvider.ts b/extensions/markdown-language-features/src/languageFeatures/documentLinkProvider.ts index 48f44da068c..650a4356f5c 100644 --- a/extensions/markdown-language-features/src/languageFeatures/documentLinkProvider.ts +++ b/extensions/markdown-language-features/src/languageFeatures/documentLinkProvider.ts @@ -271,9 +271,11 @@ export class MdLinkProvider implements vscode.DocumentLinkProvider { case 'reference': { const def = definitionSet.lookup(link.href.ref); if (def) { - return new vscode.DocumentLink( + const documentLink = new vscode.DocumentLink( link.source.hrefRange, vscode.Uri.parse(`command:_markdown.moveCursorToPosition?${encodeURIComponent(JSON.stringify([def.source.hrefRange.start.line, def.source.hrefRange.start.character]))}`)); + documentLink.tooltip = localize('documentLink.referenceTooltip', 'Go to link definition'); + return documentLink; } else { return undefined; } |