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

Changing prefix and reloading config breaks mapping #10

Closed
fiplox opened this issue Jul 22, 2024 · 5 comments
Closed

Changing prefix and reloading config breaks mapping #10

fiplox opened this issue Jul 22, 2024 · 5 comments

Comments

@fiplox
Copy link

fiplox commented Jul 22, 2024

Hi,

I'm trying to change a prefix to "`" (backtick). While it works with (key/remap :root ["ctrl+a"] ["`"]), if I reload the config with `+r, the prefix is unbound. Neither ctrl+a nor ` are working after config reload. Same with the example given in the documentation (key/remap :root ["ctrl+a"] ["ctrl+v"])

@cfoust
Copy link
Owner

cfoust commented Jul 25, 2024

Yikes! Sorry about this, I will look into it.

@fiplox
Copy link
Author

fiplox commented Jul 25, 2024

There is some other quirks I found while using it. If you want I will open separate issues, but basically:

  1. With alacritty terminal emulator cy does not respect the cursor blinking config. In my alacritty config I set
[cursor.style]
blinking = "Off"
shape = "Underline"

but in cy the cursor is blinking.
2. In alacritty while using neovim I see an annoying blinking for some time, if I understand correctly it is due to writing an output of cy to disk. With kitty it works smoothly. And overall the performance in alacritty is worse then in kitty for some reason.

P.S. Thanks for this awesome project!

@cfoust
Copy link
Owner

cfoust commented Jul 25, 2024

Interesting, thanks for the extra details. I've noticed the same behavior in alacritty; black spots also appear periodically that are not present in kitty. cy really doesn't do anything special when it comes to the output it produces, but I suspect there are some quirks in how alacritty handles cursor-related escape sequences (among other things) that are not present in kitty. I'll make an issue for these alacritty issues when I have a chance!

I hope you enjoy using cy and do let me know if you have any other issues or feedback you'd like to share! I apologize for missing this issue originally and hope to get a fix out soon.

@cfoust
Copy link
Owner

cfoust commented Jul 25, 2024

This turned out to be a bug in the trie library; I have pushed a fix to main and it will go out in the next release!

@cfoust
Copy link
Owner

cfoust commented Jul 25, 2024

Shipped in v0.3.2 and now available on brew!

@cfoust cfoust closed this as completed Jul 25, 2024
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

2 participants