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.

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

Make SelectionConverter resolve all highlighted IJavaElements #25

Open
amartya4256 opened this issue Feb 6, 2024 · 0 comments
Open
Labels
Enhancement A Request for an Enhancement of an Existing Feature Good First Issue A Good Issue to Start Eclipse Development JDT Issues for JDT

Comments

@amartya4256
Copy link

amartya4256 commented Feb 6, 2024

Issue: eclipse-jdt/eclipse.jdt.ui#1173

Current Situation

SelectionConverter can be used to highlight a piece of code and resolve it to get a context of all the IJavaElement in it. However, the method SelectionConverter.codeResolve returns an empty array if the highlighted section consists of more than one IJavaElement. This operation directly affects Self Encapsulate Fields and Create Getter and Setter operations, since the dialog opened for these operations should already have marked the highlighted fields .

Expected Behavior

SelectionConverter.codeResolve should return an array consisting of all the IJavaElement which are highlighted by the user.

Proposed Solution

TBD

@amartya4256 amartya4256 moved this to 🆕 New in Eclipse Feb 6, 2024
@HeikoKlare HeikoKlare added JDT Issues for JDT Enhancement A Request for an Enhancement of an Existing Feature Good First Issue A Good Issue to Start Eclipse Development labels Feb 6, 2024
@HeikoKlare HeikoKlare moved this from 🆕 New to 🔖 Ready: Atomic in Eclipse Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement A Request for an Enhancement of an Existing Feature Good First Issue A Good Issue to Start Eclipse Development JDT Issues for JDT
Projects
Status: 🔖 Ready: Atomic
Development

No branches or pull requests

2 participants