-
Notifications
You must be signed in to change notification settings - Fork 24
Added auto completion in the EvalOCLDialog #73
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
base: master
Are you sure you want to change the base?
Conversation
…and class members and autocompletion for use-commands (but not integrated into shell)
…e to get information about the loaded model
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.
Great: you implemented many tests 👍
Empty classes could possibly be replaced by interfaces, but this is a minor issue.
Would be great to accept suggestions by pressing <enter>
.
Reducing the suggested entries while typing would improve teh usabilty, too!
use-gui/src/main/java/org/tzi/use/autocompletion/SuggestionForClassName.java
Outdated
Show resolved
Hide resolved
}); | ||
|
||
autocompletionScrollPane = new JScrollPane(autocompletionResultList); | ||
textPane.add(autocompletionScrollPane, 1); |
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.
Is there any way to place the autocompletion windows at the position of the caret? Maybe by using another JFrame? This way it would work more like other auto completion, improving usability,
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.
Yes good idea. Since 06c5b06 the auto completion windows depend on the caret position and the prefix already present in the input text area. In my opinion, this resolves the issue effectively.
…ition and prefix. Also fixed possible null pointer in autocompletion class.
Updated documentation: CTRL+SPACE is no longer needed Removed some unnecessary comments
Both of these should work in 9739f1d as the suggestions are now dynamically updated with every written character. To move through the suggested list the ArrowDown and ArrowUp keys can be used (also see documentation/documentation.md for keybinds). |
Fixxed nullpointer when there were no suggestions Changed suggestion behaviour for = statements
May I ask how's the update going, since it's nearly a year from your last commit. |
Works for
Note: Autocompletion is currently integrated into the EvalOCLDialog only and not into the shell.
As a result, only collection operations and class members are fully supported.