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

Fails when used with dressing.nvim #5

Closed
hinell opened this issue Oct 5, 2023 · 3 comments · Fixed by #8
Closed

Fails when used with dressing.nvim #5

hinell opened this issue Oct 5, 2023 · 3 comments · Fixed by #8
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@hinell
Copy link

hinell commented Oct 5, 2023

dressing.nvim replaces native vim.ui.input.

When plugin is called, two input windows are overlaid or even replaced and nothing happens after entering.

It seems like it's impossible to have two dressing input windows at the time.

@olacin
Copy link
Owner

olacin commented Oct 8, 2023

Hi, thanks for the issue !

I don't have much bandwidth at the moment, but I am open for a PR if you're up to.

In the meantime, I would recommend you to find an alternative to dressing.nvim or to use the native vim.ui.input. 🙁

@hinell
Copy link
Author

hinell commented Oct 8, 2023

Dressing input UI may be disabled via config as workaround (doesn't affect anything else):

require("dressing").setup({
	input  = {
		 enabled = false
	}
	-- ...
})

@olacin olacin added bug Something isn't working help wanted Extra attention is needed labels Nov 24, 2023
@olacin olacin closed this as completed in #8 Nov 24, 2023
@olacin
Copy link
Owner

olacin commented Nov 24, 2023

Hey @hinell, I'm happy to share that dressing.nvim integration has been fixed and was caused by the asynchronous behavior of vim.ui.input !

Have a good one 👋

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants