Skip to content
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

Jump to bottom

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

Merged
merged 17 commits into from
Oct 12, 2020
Merged

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

merged 17 commits into from
Oct 12, 2020

Conversation

zacps
Copy link
Contributor

@zacps zacps commented Aug 30, 2020 •
edited
Loading

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
@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 marked this pull request as ready for review September 5, 2020 22:28
matklad
matklad reviewed Sep 24, 2020
View reviewed changes
@matklad
Copy link
Member

matklad commented Sep 29, 2020

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
Copy link
Contributor Author

zacps commented Oct 8, 2020

@matklad I've removed MethodOwner

I think this is probably ready now?

@matklad
Copy link
Member

matklad commented Oct 12, 2020

Looks good to me now, thanks!

bors r+

@bors
Copy link
Contributor

bors bot commented Oct 12, 2020

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

@matklad matklad matklad left review comments

@kjeremy kjeremy kjeremy left review comments

@jyn514 jyn514 jyn514 left review comments

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