search

stevearc / oil.nvim

Neovim file explorer: edit your filesystem like a buffer
MIT License
3.36k stars 91 forks source link

refactor: move loading message delay to config #424

Closed kevintraver closed 2 weeks ago

kevintraver commented 2 weeks ago

This allows loading_message_delay to be overridden

stevearc commented 2 weeks ago

In general I want to avoid exposing additional options in the config unless they are necessary. The list of options is already quite large, and it makes it hard for users to find what they're looking for. Could you describe the issue you are trying to solve with this, and why this is necessary?

kevintraver commented 2 weeks ago

I sometimes see the loading indicator flash, and would like to set it to a higher value.

Maybe this can be something thats not in config by default, but still in the docs and can be added manually.

stevearc commented 2 weeks ago

What is a value that would cause it to not flicker for you? 100ms was pulled from nowhere in particular, and I'm probably fine with any number below 500ms. I wouldn't even blink at bumping to 200ms.

kevintraver commented 2 weeks ago

Yeah, it does not flicker at 200ms.

stevearc commented 2 weeks ago

Delay updated to 200ms