Cursor blinking is triggered by changes in inactive tabs about rio HOT 5 OPEN

Resizing the window also triggers a lot of rerenders.

from rio.

Good findings, yes those behaviors should be fixed. Now cursor can come either from terminal and state, most of times comes from state, but would be good move part of cursor rendering logic outside of state (state reflects in the current window rendering state).

from rio.

Are the changes required affected by #428?

I also noticed selecting text is a lot slower when blinking-cursor is turned on.

Turned off:

Turned on:

It is not a problem when using nvim or zellij, which I am guessing is because they handle cursor state themself (?).

Edit:

Adding a check for whether there is a selection eliminates the delay after first render.

if self.state.has_blinking_enabled
	&& has_blinking_enabled
	&& self.selection_is_empty()
{
	self.context_manager.schedule_render(800);
}

I am not sure whether this is a desired long term solution, but it might make sense as a quick fix?

from rio.

Are the changes required affected by #428?

No shouldn't be affected, #428 should touch mostly sugarloaf crate.

I also noticed selecting text is a lot slower when blinking-cursor is turned on.

Hmm, that would make sense because is stacking rendering and blinking (rendering) in the same time.

Adding a check for whether there is a selection eliminates the delay after first render.

I like the solution for now but shouldn't be the final fix (I guess) because could be that users would expect the cursor to blink if have selected text (?? not sure tbh).

Would you mind create a PR with those changes?

from rio.

Sure, coming up.

from rio.