Skip to content

Fix Razor completion tooltip documentation #5839

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.

Sign up for GitHub

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

Merged
allisonchou merged 4 commits into from
Jun 21, 2023
Merged

Fix Razor completion tooltip documentation #5839

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
87e4638
Fix Razor completion documentation
allisonchou Jun 21, 2023
40ac682
Review feedback
allisonchou Jun 21, 2023
19fc69d
Review comments
allisonchou Jun 21, 2023
efcdbf2
Review comments
allisonchou Jun 21, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 20 additions & 8 deletions src/razor/src/Completion/RazorCompletionItemProvider.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ import { getUriPath } from '../UriPaths';
import { ProvisionalCompletionOrchestrator } from './ProvisionalCompletionOrchestrator';
import { LanguageKind } from '../RPC/LanguageKind';
import { RoslynLanguageServer } from '../../../lsptoolshost/roslynLanguageServer';
import { CompletionItem, CompletionParams, CompletionTriggerKind } from 'vscode-languageclient';
import { CompletionItem, CompletionList, CompletionParams, CompletionTriggerKind, MarkupContent } from 'vscode-languageclient';
import { UriConverter } from '../../../lsptoolshost/uriConverter';
import * as RazorConventions from '../RazorConventions';
import { MappingHelpers } from '../Mapping/MappingHelpers';
Expand All @@ -35,11 +35,6 @@ export class RazorCompletionItemProvider

let completions: vscode.CompletionList | vscode.CompletionItem[];

// For CSharp, completions need to keep the "data" field
// on the completion item for lazily resolving the edits in
// the resolveCompletionItem step. Using the vs code command
// drops that field because it doesn't exist in the declared vs code
// CompletionItem type.
if (language === LanguageKind.CSharp) {
const params: CompletionParams = {
context: {
Expand All @@ -52,6 +47,11 @@ export class RazorCompletionItemProvider
position: projectedPosition
};

// For CSharp, completions need to keep the "data" field on the
// completion item for lazily resolving the edits in the
// resolveCompletionItem step. Using the vs code command drops
// that field because it doesn't exist in the declared vs code
// CompletionItem type.
completions = await vscode
.commands
.executeCommand<vscode.CompletionList | vscode.CompletionItem[]>(
Expand All @@ -71,6 +71,8 @@ export class RazorCompletionItemProvider
completions instanceof Array ? completions // was vscode.CompletionItem[]
: completions ? completions.items // was vscode.CompletionList
: [];

const data = (<CompletionList>completions)?.itemDefaults?.data;

// There are times when the generated code will not line up with the content of the .razor/.cshtml file.
// Therefore, we need to offset all completion items' characters by a certain amount in order
Expand All @@ -79,7 +81,7 @@ export class RazorCompletionItemProvider
const completionCharacterOffset = projectedPosition.character - hostDocumentPosition.character;
for (const completionItem of completionItems) {
const doc = completionItem.documentation as vscode.MarkdownString;
if (doc) {
if (doc && doc.value) {
// Without this, the documentation doesn't get rendered in the editor.
const newDoc = new vscode.MarkdownString(doc.value);
newDoc.isTrusted = doc.isTrusted;
Expand Down Expand Up @@ -128,6 +130,10 @@ export class RazorCompletionItemProvider
completionItem.insertText = intellicodeCompletion.textEditText;
}
}

if (!(<CompletionItem>completionItem).data) {
(<CompletionItem>completionItem).data = data;
}
}

const isIncomplete = completions instanceof Array ? false
Expand Down Expand Up @@ -204,6 +210,13 @@ export class RazorCompletionItemProvider

item = newItem;

// The documentation object Roslyn returns is a MarkupContent,
// which we need to convert to a MarkdownString.
const markupContent = <MarkupContent>(<unknown>(item.documentation));
if (markupContent && markupContent.value) {
item.documentation = new vscode.MarkdownString(markupContent.value);
}

if (item.command && item.command.arguments?.length === 4) {
let uri = vscode.Uri.parse(item.command.arguments[0]);

Expand Down Expand Up @@ -238,4 +251,3 @@ function getTriggerKind(triggerKind: vscode.CompletionTriggerKind): CompletionTr

}
}