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

Go to definition feature is needed #5

Closed
buggins opened this issue Jan 29, 2015 · 7 comments · Fixed by #29
Closed

Go to definition feature is needed #5

buggins opened this issue Jan 29, 2015 · 7 comments · Fixed by #29
Assignees
@buggins
Labels
enhancement

Comments

@buggins
Copy link
Owner

buggins commented Jan 29, 2015

No description provided.

@buggins buggins self-assigned this Jan 29, 2015
@Freakazo
Copy link
Contributor

Would it be possible to use something like dscanner?

Freakazo added a commit to Freakazo/dlangide that referenced this issue Feb 4, 2015
@Freakazo
Tackle issue buggins#5.
d09841d 
Added: Application looks for ctrl+click, starts "dcd-client -l -c "
with the line number and file name. If response is a file, open it.
@Freakazo
Copy link
Contributor

Freakazo commented Feb 4, 2015

Hi, I tried to add this feature. See the referenced commit above. Just to be clear that commit is for reference and not a pull request.

It works by looking for a control+click and then sends the location to DCD then placing the caret at that position if it's in the same file, otherwise it tries to open it. All implemented in a very messy proof of concept way.

What would the dlang-ide way be to implement something like this? I'm almost completely lost in this regards.

@buggins
Copy link
Owner Author

buggins commented Feb 10, 2015

Thank you! Let me check your implementation.

@Freakazo
Copy link
Contributor

I've started work on a much nicer, cleaner and generalized solution. I will post up in a few days :)

@buggins
Copy link
Owner Author

buggins commented Feb 10, 2015

Would be great. Thanks :)

@extrawurst extrawurst mentioned this issue Feb 10, 2015
Closed
@extrawurst
Copy link
Contributor

@Freakazo will you implement it in a way that can be used to tackle issue #20 "Code Completion using DCD" too ?

@Freakazo
Copy link
Contributor

Yes, I am.

Freakazo added a commit to Freakazo/dlangide that referenced this issue Feb 11, 2015
@Freakazo
Initial implementation of "Go to definition".
d766508 
Uses DCD to get location. See issue buggins#5
@Freakazo Freakazo mentioned this issue Feb 14, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@buggins buggins

Labels
enhancement
Projects
None yet
Milestone
No milestone
3 participants
@buggins @extrawurst @Freakazo