Spring/spring-shell
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Restore cursor using normal capability

Browse Source 
- Looks like some infocmp capabilities for making cursor visible
  using cvvis doesn't always work. So change to use cnorm instead.
- Terminal instruction of making cursor very visible with cvvis
  doesn't mean cursor should be visible, afaik, so i.e. screen
  or tmux didn't work.
- Fixes #970
This commit is contained in:
Janne Valkealahti
2024-01-12 15:41:46 +00:00
parent 1f027b8a7d
commit a6d16e8cbb
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
spring-shell-core/src/main/java/org/springframework/shell/component/support/AbstractComponent.java
View File

@@ -294,7 +294,7 @@ public abstract class AbstractComponent<T extends ComponentContext<T>> implement
finally {
terminal.setAttributes(attr);
terminal.puts(Capability.keypad_local);
terminal.puts(Capability.cursor_visible);
terminal.puts(Capability.cursor_normal);
display.update(Collections.emptyList(), 0);
}
}

4
spring-shell-core/src/main/java/org/springframework/shell/component/view/TerminalUI.java
View File

@@ -283,7 +283,7 @@ public class TerminalUI implements ViewService {
int targetCursorPos = 0;
if (virtualDisplay.isShowCursor()) {
terminal.puts(Capability.cursor_visible);
terminal.puts(Capability.cursor_normal);
targetCursorPos = size.cursorPos(virtualDisplay.getCursorPosition().y(), virtualDisplay.getCursorPosition().x());
log.debug("Display targetCursorPos {}", targetCursorPos);
}
@@ -402,7 +402,7 @@ public class TerminalUI implements ViewService {
eventLoop.destroy();
terminal.setAttributes(attr);
log.debug("Setting cursor visible");
terminal.puts(Capability.cursor_visible);
terminal.puts(Capability.cursor_normal);
if (fullScreen) {
display.update(Collections.emptyList(), 0);
}