-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Autocomplete++ #2236
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++ #2236
Conversation
Release EnvironmentsThis Environment is provided by Release, learn more! 🔧Environment Status : https://app.release.com/public/Processing%20Foundation/env-854911fa77 |
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.
Thanks for these updates, I think the altogether the autocomplete, argument previews, and link to references make this really helpful and amazing!
Wow, impressive, @peilingjiang! Side question (perhaps for @raclim). |
@Atcold The PR has been merged, but we haven't pushed it out live to production yet due to some bug fixes and additional testing we'd like to perform before distributing it to everyone, which we're planning to complete by the end of June. I'm hoping to provide a more consistent way to communicate when these releases will happen, but for now I'll definitely send an update when it's out live! |
Hi, just wanted to update that this is out live on production! :) I made one minor change, where I set the initial preference for the Autocomplete and Hinter to 'Off' instead of 'On', since I felt this feature might be a little confusing when a new user is encountering the editor for the very first time. However, please let me know if you feel like this should be changed or have a different suggestion, thanks again for all of your work/input on this! |
I have some thoughts: for those who have an account and some works, they may be familiar with the online editor. Then they may not be so confusing. So making this option on for them is a boost. For newbies: since now there are a lot of web editors based on https://github.com/microsoft/monaco-editor, and it has auto-complete out of the box. (See https://swc.rs/playground for example). I guess even a newbie can handle it. |
I agree with @wuyudi. New users would benefit from having suggestions out of the box. On the other side, if one dislikes the functionality (for whatever reason), one can look for ways to stop it. In short, one doesn't know one doesn't know. |
Fixes #99 and extends #1873
Changes: Extending the current dropdown hinter solution by adding a Copilot-style inline completion preview, which includes argument previews for the functions.
I have verified that this pull request:
npm run lint
)npm run test
)develop
branch.Fixes #123