Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Switching from alternative screen to normal screen in the terminal working incorrect after resize #4315

Closed
AndrienkoAleksandr opened this issue Mar 6, 2017 · 4 comments
Closed

Switching from alternative screen to normal screen in the terminal working incorrect after resize #4315

AndrienkoAleksandr opened this issue Mar 6, 2017 · 4 comments
Assignees
@AndrienkoAleksandr
Labels
kind/bug Outline of a bug - must adhere to the bug report template. severity/P2 Has a minor but important impact to the usage or development of the system.

Comments

@AndrienkoAleksandr
Copy link
Contributor

AndrienkoAleksandr commented Mar 6, 2017 •
edited
Loading

Steps to reproduce:

  1. Open terminal.
  2. Type mc and Enter.
  3. Resize terminal.
  4. Type Ctrl + O.
    Actual result: mc output still exist.
    Expected result: terminal should delete mc output (output alternative screen).
    Current terminal (and previous old terminal) wrong switch from alternative to normal screen. Issue related to Resizing the terminal while in the alt buffer will break the normal buffer xtermjs/xterm.js#510
@AndrienkoAleksandr AndrienkoAleksandr added kind/task Internal things, technical debt, and to-do tasks to be performed. team/enterprise labels Mar 6, 2017
@AndrienkoAleksandr AndrienkoAleksandr self-assigned this Mar 6, 2017
@AndrienkoAleksandr AndrienkoAleksandr added the status/in-progress This issue has been taken by an engineer and is under active development. label Mar 6, 2017
@AndrienkoAleksandr AndrienkoAleksandr mentioned this issue Mar 6, 2017
Closed
@AndrienkoAleksandr AndrienkoAleksandr removed the status/in-progress This issue has been taken by an engineer and is under active development. label Mar 14, 2017
@AndrienkoAleksandr AndrienkoAleksandr changed the title Rework(Improve) mechanizm switch off alternative screen and normal screen in the terminal Switching from alternative screen to normal screen in the terminal working incorrect after resize Mar 16, 2017
@AndrienkoAleksandr AndrienkoAleksandr added kind/bug Outline of a bug - must adhere to the bug report template. and removed kind/task Internal things, technical debt, and to-do tasks to be performed. labels Mar 16, 2017
@vkuznyetsov vkuznyetsov added the severity/P2 Has a minor but important impact to the usage or development of the system. label Apr 19, 2017
@AndrienkoAleksandr
Copy link
Contributor Author

Can be fixed if issue #5084 will be resolved.

@ghost
Copy link

ghost commented Jul 12, 2017

Still an issue in 5.15-SNAPSHOT

@AndrienkoAleksandr
Copy link
Contributor Author

This bug was fixed for xterm.js since vesion 2.9.1. But this version contains quite critical bug xtermjs/xterm.js#885, so I don't think we can update terminal while this bug is not fixed...

@AndrienkoAleksandr AndrienkoAleksandr added the status/blocked Issue that can’t be moved forward. Must include a comment on the reason for the blockage. label Aug 15, 2017
@AndrienkoAleksandr AndrienkoAleksandr mentioned this issue Nov 14, 2017
Closed
@skabashnyuk skabashnyuk removed the status/blocked Issue that can’t be moved forward. Must include a comment on the reason for the blockage. label Jun 27, 2018
@AndrienkoAleksandr
Copy link
Contributor Author

Fixed after update xterm.js to version 3.3.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AndrienkoAleksandr AndrienkoAleksandr

Labels
kind/bug Outline of a bug - must adhere to the bug report template. severity/P2 Has a minor but important impact to the usage or development of the system.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@skabashnyuk @tolusha @vkuznyetsov @AndrienkoAleksandr