folke / noice.nvim

💥 Highly experimental plugin that completely replaces the UI for messages, cmdline and the popupmenu.
Apache License 2.0
3.95k stars 88 forks source link

fix(hacks): remove redundancies in `show_cursor` #870

Open ttytm opened 1 week ago

ttytm commented 1 week ago

I was constantly running into the issue of my cursor disappearing. To get it back, I had to manually(or via a dedicated au) re-set the gcr/guicursor setting.

Checking the code, I see you have some redundant checks for the same condition. if M._guicursor and if not Util.is_exiting() are both check inside and outside of the scheduled function.

The removed double checks should also result in a minor improvement reducing load.

This fix can also help with the reports that people still experience #114.