Skip to content

Add a command to open docs for the symbol under the cursor#5917

Merged
bors[bot] merged 17 commits into
rust-lang:masterfrom
zacps:open-documentation-under-cursor
Oct 12, 2020
Merged

Add a command to open docs for the symbol under the cursor#5917
bors[bot] merged 17 commits into
rust-lang:masterfrom
zacps:open-documentation-under-cursor

Conversation

@zacps

@zacps zacps commented Aug 30, 2020 •
edited
Loading

Copy link
Copy Markdown
Contributor

Todo

  • Decide if there should be a default keybind or context menu entry
  • Figure out how to get the documentation path for methods and other non-top-level defs
  • Design the protocol extension. In future we'll probably want parameters for local/remote documentation URLs, so that should maybe be done in this PR?
  • Code organisation
  • Tests

@zacps zacps force-pushed the open-documentation-under-cursor branch from d4b4d2f to b1427c3 Compare August 31, 2020 08:33
matklad
matklad reviewed Aug 31, 2020
View reviewed changes
Comment thread crates/rust-analyzer/src/lsp_ext.rs Outdated
Comment thread crates/rust-analyzer/src/lsp_ext.rs Outdated
Comment thread crates/rust-analyzer/src/lsp_ext.rs Outdated
Comment thread crates/rust-analyzer/src/lsp_ext.rs Outdated
Comment thread crates/rust-analyzer/src/lsp_ext.rs
Comment thread editors/code/src/lsp_ext.ts Outdated
Comment thread crates/rust-analyzer/src/handlers.rs Outdated
Comment thread crates/ide/src/lib.rs Outdated
Comment thread crates/hir/src/doc_links.rs Outdated
@zacps zacps force-pushed the open-documentation-under-cursor branch 3 times, most recently from 09982d8 to c2aa141 Compare September 5, 2020 05:27
zacps
zacps commented Sep 5, 2020
View reviewed changes
Comment thread crates/ide/src/doc_links.rs
@zacps zacps marked this pull request as ready for review September 5, 2020 22:28
matklad
matklad reviewed Sep 24, 2020
View reviewed changes
Comment thread crates/hir/src/code_model.rs Outdated
Comment thread crates/hir/src/code_model.rs Outdated
Comment thread crates/hir_def/src/item_tree.rs
Comment thread crates/ide/src/doc_links.rs Outdated
@matklad

matklad commented Sep 29, 2020

Copy link
Copy Markdown
Contributor

Status: MethodOwner enum should be removed from hir, and the "let's consider only adt's" logic should be moved to ide layer. Other than that, LGTM!

zacps added 16 commits October 8, 2020 14:59
@zacps
WIP: Command to open docs under cursor
bfda0d2
@zacps
Add support for struct & trait methods
a06d736
@zacps
Rename ide::link_rewrite -> ide::doc_links & tidy imports
8c32bde
@zacps
Changes from review
a14194b
@zacps
Code reorganisation and field support
974518f
@zacps
Document the protocol extension
62b76e7
@zacps
Differentiate method/tymethod by determining 'defaultness'
c648884 
Currently a method only has defaultness if it is a provided trait
method, but this will change when specialisation is available and may
need to become a concept known to hir.

I opted to go for a 'fewest changes' approach given specialisation is
still under development.
@zacps
Change Option::Some bug to a fixme note
26086fa 
IMO this is too much work to be worth fixing at the moment.
@zacps
Fix send->sent typo
f1decfc
@zacps
Update tests for new function field
ec75d8b
@zacps
Add tests
37a4d06
@zacps
Fix namespace detection & function test
6cae6b8
@zacps
Add ignored test to demonstrate ImportMap bug
f6759ba
@zacps
Remove outdated part of doc_links module docs
e4a787f
@zacps
Changes from review
d2c6880
@zacps
Rebase fixes
8af1dd7
@zacps zacps force-pushed the open-documentation-under-cursor branch from 40747a0 to 8af1dd7 Compare October 8, 2020 02:23
@zacps zacps force-pushed the open-documentation-under-cursor branch from dfde80e to 3bd4fe9 Compare October 8, 2020 02:50
@zacps

zacps commented Oct 8, 2020

Copy link
Copy Markdown
Contributor Author

@matklad I've removed MethodOwner

I think this is probably ready now?

@matklad

matklad commented Oct 12, 2020

Copy link
Copy Markdown
Contributor

Looks good to me now, thanks!

bors r+

@bors

bors Bot commented Oct 12, 2020

Copy link
Copy Markdown
Contributor

Build succeeded:

@bors bors Bot merged commit 518f6d7 into rust-lang:master Oct 12, 2020
@fannheyward fannheyward mentioned this pull request Oct 12, 2020
Open
@benknoble benknoble mentioned this pull request Oct 30, 2020
Open
@syphar syphar mentioned this pull request May 2, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

3 more reviewers

@matklad matklad matklad left review comments

@kjeremy kjeremy kjeremy left review comments

@jyn514 jyn514 jyn514 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@zacps @matklad @kjeremy @jyn514