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

Allow to disable asm auto-completion #29

Closed
Dark-Schneider-666 opened this issue Apr 2, 2023 · 12 comments
Closed

Allow to disable asm auto-completion #29

Dark-Schneider-666 opened this issue Apr 2, 2023 · 12 comments
Labels
Priority: High Priority: High Type: Enhancement New feature or request

Comments

@Dark-Schneider-666
Copy link

Hi,

I feel this plug-in almost perfect but for the asm auto-completion, IMHO very intrusive. The asm language is "short" to write, the completion boxes appearing so frequently for short writes can be distracting, and sometimes you have to press even more keys, i.e. when writing "ret", have to press enter 2 times for going to the next line.
Then I see very useful the auto-completion of labels, but not so much for the asm code itself. If I could disable it I'd do.

Thanks for the plug-in to all this.

@theNestruo
Copy link
Contributor

Hi,

I think the same about auto-completion in assembly, but it can be solved with language-specific configuration in the user settings (settings.json):

"[z80-macroasm]": {
	"editor.quickSuggestions": {
		"comments": "off",
		"strings": "off",
		"other": "off"
	},
},

Note: I also have this two settings in general (i.e.: for any language). I think they are not relevant, but, just in case...:

"editor.suggestOnTriggerCharacters": false,
"editor.wordBasedSuggestions": false,

@Dark-Schneider-666
Copy link
Author

Hello,
Tried but the auto-completion is still there even after reload.

@mborik
Copy link
Owner

mborik commented Apr 3, 2023

Hi @Dark-Schneider-666,
thank you for your very good and relevant suggestion. Honestly, last 2 months I've been doing a demo for FOReVER demoparty (and it won!!! 🤩) and I was seriously pi**ed off with my own extension for exactly same reason. So, when I manage to finish my current WIP projects in next few weeks I will implement such configuration options to disable suggests on something, like us, who are highly skilled in Z80 don't need. 👍

@mborik mborik added Type: Enhancement New feature or request Priority: High Priority: High labels Apr 3, 2023
@mborik
Copy link
Owner

mborik commented May 3, 2023

Hi @Dark-Schneider-666, version v0.7.9 which I just released should not suggest instructions by default. Please, let me know if it improve your experience. 😉

@anrikun
Copy link

anrikun commented May 4, 2023

Nice suggestion and addition @Dark-Schneider-666 @mborik! I'll test it this weekend as well.

@ssg
Copy link

ssg commented May 4, 2023

I just hit that problem yesterday when I wanted to give the extension a spin. It's especially problematic when you write ld hl, and autocomplete appears but you habitually press space, and it completes it to ld hl,a. Quite annoying. Let me give the new version a try as well. :)

@ssg
Copy link

ssg commented May 4, 2023

It seems like I have the latest version and still hit that problem despite "Suggest On Instructions" is disabled. I guess I need "argument suggestions" to be turned off too.

@ssg
Copy link

ssg commented May 4, 2023

@theNestruo's "editor.suggestOnTriggerCharacters": false setting recommendation fixed that problem for me.

@Dark-Schneider-666
Copy link
Author

Dark-Schneider-666 commented May 5, 2023

@theNestruo's "editor.suggestOnTriggerCharacters": false setting recommendation fixed that problem for me.

Nice. As I had it installed from earlier it was not set by default even if uninstalled and reinstalled as VSCode saves the config and didn't know where it saves on macOS (have to find).

@mborik
Copy link
Owner

mborik commented May 6, 2023

I just hit that problem yesterday when I wanted to give the extension a spin. It's especially problematic when you write ld hl, and autocomplete appears but you habitually press space, and it completes it to ld hl,a. Quite annoying. Let me give the new version a try as well. :)

Ah, I see! Thank you for this specific case description. I think that issue is, that one of commit characters for instructions and arguments completion is space. I think that could lead to these problems. This would be easy to fix. ☝️

mborik added a commit that referenced this issue May 9, 2023
@mborik
Copy link
Owner

mborik commented May 9, 2023

Hi guys, especially @ssg, check latest release, please, where I removed space key as commit character for suggestions.

@mborik mborik closed this as completed Jun 7, 2023
@ssg
Copy link

ssg commented Aug 1, 2023

sorry @mborik, many things have come up at the same time, so I couldn't check it yet. I'll comment here if it repros in the latest version. thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: High Priority: High Type: Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants