Sweep binds "Tab" to autocompletion regardless of whether it's bound in the settings

[leaden marten]

I like to use another shortcut for autocompletion, but when removing Tab from the "Accept Autocomplete Suggestion" and setting another shortcut, hitting Tab will still trigger Sweep autocompletion no matter what (the hotkey I set will also work, but not only it). It seems to be bound to the "Tab" key itself as defined in the IDE's keymap settings - if I remove that binding, it no longer autocompletes with it (but then I can't use Tab to indent at all anymore, so I can't do that).

Would it be possible for Sweep to exclusively respond to the "Accept Autocomplete Suggestion" shortcut?

Related bonus (albeit much less important) request: could this placeholder be changed to include the (first) key bound to the "Accept Autocomplete Suggestion" as well, instead of having "Tab" (seemingly) hardcoded?

Thanks!

[tired thunder]

Hi Jeto! We'll take a look at this. Thank you for the feedback!

[leaden marten]

Sorry to bump this - I know you guys are busy improving the plugin seemingly every day already.

Many times a day I hit Tab hoping to indent or perform basic autocompletion / apply live templates (e.g. by writing log and then immediately hitting Tab to print console.log(<cursor>), however doing so often triggers Sweep autocompletion instead, making it very random and a bit frustrating.

As a dev I assume (maybe wrongly!) that this shouldn't be a big fix as it probably means the action is hard-bound to Editor Actions > Tab instead of being only bound to Sweep's Accept Autocomplete Suggestion (which should default to the Tab keyboard button).

Let me know if I can help with any additional info!

[normal creek]

No problem, we will raise this in priority!

[odd helm]

+1 to this one. Not being able to unbind tab from sweep completion is uncomfortable.

[tired thunder]

Thank you for bumping this, we'll try to get it out this week!

[normal creek]

@leaden marten @odd helm Would you mind telling me the keybindings you would prefer? Asking so I can make sure that I test these to make sure it works properly

[leaden marten]

Hey! Logically it should be whatever Accept Autocomplete Suggestion is bound to (which is the Tab key by default), but it should not also be bound to Editor Actions > Tab which is a specific binding for indenting code. Right now it's bound to both of those action.

In my screenshots you can see my current bindings - I would expect only ⌥ + C to autocomplete here, yet ⇥ (tab) also does.

So there shouldn't be a need to change any key binding, but just have Sweep honor the relevant action and only it.

Hope this makes sense, lmk if it doesn't.

[tired thunder]

We will share an early build, would you be open to helping test it for us?

[leaden marten]

Of course 👍

[normal creek]

Ill share it with you in a bit!

[normal creek]

@leaden marten Here is the build. If you use GateWay please don't install this as it currently breaks on GateWay and I am in the process of fixing the issues.
Settings > Plugins > Install Plugin from Disk

Please let me know if you run into any issues!

[odd helm]

Thank you so much! Tried it, and yep, works great! I usually use shift+enter btw.

[leaden marten]

Can confirm, this works perfectly. Thanks!

PS: The badge (if enabled) still reads "Tab" (ideally it should be based on the first bound hotkey), but honestly that can totally wait until a future update as it's purely cosmetic

[normal creek]

This will be fixed in the official release 1.26.5 which should be out soon (after JetBrains approves the release)

[leaden marten]

Thanks! Already tried it with William's version from another thread, looks perfect - although this one is still looks incorrect (but again, really low priority of course!)