Skip to content

Allow MarkupContent[] to be used as Hover contents#572

Closed
DJMcNab wants to merge 1 commit intomicrosoft:gh-pagesfrom
DJMcNab:gh-pages
Closed

Allow MarkupContent[] to be used as Hover contents#572
DJMcNab wants to merge 1 commit intomicrosoft:gh-pagesfrom
DJMcNab:gh-pages

Conversation

@DJMcNab
Copy link

@DJMcNab DJMcNab commented Sep 23, 2018

@DJMcNab
Copy link
Author

DJMcNab commented Sep 24, 2018

Interestingly, atom-languageclient already supports this functionality:

https://github.com/atom/atom-languageclient/blob/606faa36216657cfd8e1cea9cd2b7974e3a357ce/lib/adapters/datatip-adapter.ts#L54-L56

DJMcNab added a commit to DJMcNab/vscode-languageserver-node that referenced this pull request Nov 21, 2018
@DJMcNab
Copy link
Author

DJMcNab commented Nov 21, 2018

Closing after discussion with @dbaeumer in microsoft/vscode-languageserver-node#417. Note that atom-languageclient does not use this same approach (at least when I last checked.

Sourcegraph uses css rules to make the markdown content with horizontal seperators look the same as multiple array items: sourcegraph/codeintellify@2ab2bf6

@DJMcNab DJMcNab closed this Nov 21, 2018
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