Navigate to variable declaration from variable view#642
Merged
SougandhS merged 1 commit intoeclipse-jdt:masterfrom Mar 24, 2025
Merged
Conversation
Contributor
Author
|
@SougandhS could you please review this PR? |
Member
|
Please update the PR description and steps to test |
SougandhS
requested changes
Mar 17, 2025
SougandhS
reviewed
Mar 17, 2025
Contributor
Author
Done |
Member
|
Hi @subyssurendran666 ,On testing the PR i found 3 issues
|
subyssurendran666
force-pushed
the
navigate_to_local_variable_while_debug
branch
from
77b8f74 to
7752d38
Compare
Contributor
Author
These three scenarios handled. |
SougandhS
requested changes
Mar 21, 2025
subyssurendran666
force-pushed
the
navigate_to_local_variable_while_debug
branch
from
7752d38 to
c2b44cd
Compare
SougandhS
requested changes
Mar 24, 2025
SougandhS
reviewed
Mar 24, 2025
subyssurendran666
force-pushed
the
navigate_to_local_variable_while_debug
branch
from
c2b44cd to
523b086
Compare
SougandhS
approved these changes
Mar 24, 2025
Member
SougandhS
left a comment
SougandhS
left a comment
There was a problem hiding this comment.
Please squash all commits to one.
This commit adds context menu in variables view to highlight variable's declaration in the source editor. Fixes : eclipse-jdt#640
subyssurendran666
force-pushed
the
navigate_to_local_variable_while_debug
branch
from
523b086 to
f08f600
Compare
Member
|
Thank you @subyssurendran666 for your contribution ✨ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What it does
All necessary discussions so far have been added to the parent ticket.
This PR closes #640
How to test
Steps
Right click on the variable in the variables view while on debug mode. A new option have beed added("Open local variable Navigation)
2.
Once it selected, the selected variable highlighted.
3.
The "Open local variable navigation" option would not available for unwanted items.
Author checklist