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

Improve visibility of auto-completion shortcut #8

Closed
tferr opened this issue Feb 18, 2022 · 8 comments
Closed

Improve visibility of auto-completion shortcut #8

tferr opened this issue Feb 18, 2022 · 8 comments

Comments

@tferr
Copy link
Collaborator

tferr commented Feb 18, 2022

As mentioned in the forum, it is not obvious that auto-completion requires Ctrl+Space. Also, IJM auto-completion does not require it. We should make things more obvious and consistent, by making the shortcut more obvious in the GUI, and by making it optional.

@haesleinhuepf, @ctrueden, @imagejan: Do you know how the IJM auto-completion bypasses Ctrl+Space?

@imagesc-bot
Copy link

This issue has been mentioned on Image.sc Forum. There might be relevant details there:

https://forum.image.sc/t/jython-auto-completion-of-fiji-methods-in-eclipse-ide/63103/10

@imagesc-bot
Copy link

This issue has been mentioned on Image.sc Forum. There might be relevant details there:

https://forum.image.sc/t/cant-release-project-script-editor/63240/30

@haesleinhuepf
Copy link
Member

@haesleinhuepf, @ctrueden, @imagejan: Do you know how the IJM auto-completion bypasses Ctrl+Space?

I implemented that to mimic IntelliJ-behaviour: https://github.com/imagej/imagej-legacy/blob/master/src/main/java/net/imagej/legacy/plugin/MacroLanguageSupportPlugin.java#L165

@tferr
Copy link
Collaborator Author

tferr commented Feb 18, 2022

I did not know that! Neat! To me, Ctrl+Space is muscle memory because that's what I'm used to in Eclipse. How do you suggest handling this? Shall we implement the same on the jython side (thus disabling the need for the shortcut combo), or shall it become a formal option somewhere in the GUI? If the latter, I would really need some loose pointers on where to implement that.

pinging @acardona for opinion

@haesleinhuepf
Copy link
Member

I presume the Eclipse-based developers will have this opinion and IntelliJ-based developers will have that opinion.

I implemented the auto-completion in a way that it jumps into the face of macro-programmers intentionally. I assume if it wasn't like this, the majority of users would have never discovered that auto-completion exists.

Having an option to switch it to press-CTRL+space-to-auto-complete makes sense though. It could live right next to the checkbox that turns auto-completion completely off:

https://github.com/scijava/script-editor/blob/546062a25625da0cb34dd51af9b98105f7f82ace/src/main/java/org/scijava/ui/swing/script/TextEditor.java#L1463

@acardona
Copy link
Collaborator

acardona commented Feb 20, 2022

Control+space works, can't add these new features: instead, I spent all my time trying, and failing, to release other features. The mvn build system is broken to the bone is some fundamentally horrible way, and now it stole the time I had to actually add value.

If you think what I am saying is hyperbole, see the last events on the saga: now even Fiji.app downloaded from fiji.sc doesn't launch. And when it does after removing the offending jar file, then the updater freezes. I mean one can't make this stuff up. https://forum.image.sc/t/cant-release-project-script-editor/63240/40?u=albertcardona

@imagejan
Copy link
Member

@haesleinhuepf wrote:

Having an option to switch it to press-CTRL+space-to-auto-complete makes sense though.

Absolutely. I suggested this a while ago here: imagej/imagej-legacy#254

@tferr
Copy link
Collaborator Author

tferr commented Feb 24, 2022

Closing this, as it is addressed by PR#56

@tferr tferr closed this as completed Feb 24, 2022
ctrueden pushed a commit to scijava/script-editor that referenced this issue Feb 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants