Thank you for your PR and detailed explanation!

It seems that for users whose default cursor is not a block, set_cursor_block doesn't make sense.

I think it's best to change set_cursor_block to set_cursor_default and use SetCursorStyle::DefaultUserShape instead of SetCursorStyle::BlinkingBlock here.

You're right! It makes more sense, I updated it

There seems one missed

Okay I'll update this one too, but I'm not sure that we should show default shape in normal mode, if user's default shape is bar, then this might be confusing (same shape in both normal and insert modes)?

Oh! I didn't think of that...

Keeping it as a block in normal mode should make sense, and it should restore to the default upon exiting.

Thank you, merged!

I think this issue persists when confirming There are … tasks running, sure to quit? (y/N) to exit the program. I didn't look at the code, but maybe that code path avoids the fix.