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
autocomplete for build.properties #508
Conversation
A rudimentary implementation of the auto-complete feature for the build.properties file editor for features, plugins and fragments. Searches and auto-fills valid property names. Fixes: eclipse-pde#503
a6b831d
to
71ed9cb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, you might want to add a screenshot here how it looks like when auto-completion is active.
I'm adding @mickaelistria as a reviewer as he is more familiar with all the autocomplete/editor stuff and might has some improvements or hints. |
thanks @laeubi . auto-fill popup when content assist key is pressed on a half-baked property: |
Very cool, thanks @gireeshpunathil |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @gireeshpunathil for this, it is a handy addition and sorry for the late silence.
Although this is already merged I added some remarks below. Maybe you want to address them in a follow up?
.../org/eclipse/pde/internal/ui/editor/contentassist/BuildPropertiesContentAssistProcessor.java
Show resolved
Hide resolved
.../org/eclipse/pde/internal/ui/editor/contentassist/BuildPropertiesContentAssistProcessor.java
Show resolved
Hide resolved
.../org/eclipse/pde/internal/ui/editor/contentassist/BuildPropertiesContentAssistProcessor.java
Show resolved
Hide resolved
...ipse.pde.ui/src/org/eclipse/pde/internal/ui/editor/build/BuildSourceViewerConfiguration.java
Show resolved
Hide resolved
a few review comments are included in this follow-up PR: - removed inheriting from ICompletionListener - hence removed completion listener registration (ICompletionListener.addCompletionListener) - usage of array creation shortcut - better string initialization - localization of fContentAssistantProcessor variable Refs: eclipse-pde#508
a few review comments are included in this follow-up PR: - removed inheriting from ICompletionListener - hence removed completion listener registration (ICompletionListener.addCompletionListener) - usage of array creation shortcut - better string initialization - localization of fContentAssistantProcessor variable Refs: eclipse-pde#508
@HannesWell - #529 , thanks in advance! |
a few review comments are included in this follow-up PR: - removed inheriting from ICompletionListener - hence removed completion listener registration (ICompletionListener.addCompletionListener) - usage of array creation shortcut - better string initialization - localization of fContentAssistantProcessor variable Refs: #508
A rudimentary implementation of the auto-complete feature for the build.properties file editor for features, plugins and fragments. Searches and auto-fills valid property names.
Pending items would be to handle corner cases (if any), provide a help text, provide potential suggestions for the values. But I prefer to handle one item at a time - I am still learning things!
Fixes: #503