This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Update clangd-vscode docs to be more user-focused.
ClosedPublic

Authored by sammccall on Sep 3 2019, 4:12 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

sammccall created this revision.Sep 3 2019, 4:12 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 3 2019, 4:12 AM
kadircet accepted this revision.Sep 3 2019, 4:25 AM

LGTM, thanks for working on this!

clang-tools-extra/clangd/clients/clangd-vscode/README.md
5 ↗(On Diff #218431)

can we also have links to sections below?

9 ↗(On Diff #218431)

what about code-actions(and also some screenshots)

This revision is now accepted and ready to land.Sep 3 2019, 4:25 AM
sammccall updated this revision to Diff 218436.Sep 3 2019, 4:49 AM

describe refactoring features

hokein added a comment.Sep 3 2019, 4:49 AM

could we move the png files to a separate directory like clangd/clients/clangd-vscode/assets?

sammccall updated this revision to Diff 218437.Sep 3 2019, 4:52 AM

mention rename

sammccall marked 3 inline comments as done.Sep 3 2019, 4:53 AM
sammccall added inline comments.
clang-tools-extra/clangd/clients/clangd-vscode/README.md
5 ↗(On Diff #218431)

The intent here is a summary rather than a TOC, and that the page is short enough that the first is already visible and the others would jump down only a screen or so. I think the links would distract more than help.

9 ↗(On Diff #218431)

Added a section for refactoring. (code-actions is a mechanism, but not really a user-visible feature I think).

kadircet added inline comments.Sep 3 2019, 4:54 AM
clang-tools-extra/clangd/clients/clangd-vscode/README.md
9 ↗(On Diff #218431)

thanks, I was implying refactorings

95 ↗(On Diff #218437)

s/rerame/rename

sammccall updated this revision to Diff 218438.Sep 3 2019, 4:59 AM
sammccall marked an inline comment as done.

move images into doc-assets/

sammccall updated this revision to Diff 218444.Sep 3 2019, 6:27 AM

Fix mailing list link again

hokein accepted this revision.Sep 3 2019, 7:25 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptSep 9 2019, 4:37 AM