Skip to content

Conversation

@jerch
Copy link
Member

@jerch jerch commented Aug 4, 2019

Basic SL / SR / DECIC / DECDC support.

TODO:

  • test files against xterm behavior
  • implement DECLRMM
  • test interaction with DECLRMM

Update: DECLRMM should go into a separate PR as its more involved (also influences other VT primitives).

@jerch jerch added the work-in-progress Do not merge label Aug 4, 2019
@jerch jerch self-assigned this Aug 4, 2019
@jerch jerch changed the title Left/right scroll support column sequences Aug 4, 2019
@jerch jerch mentioned this pull request Sep 3, 2019
11 tasks
@jerch jerch removed the work-in-progress Do not merge label Oct 24, 2019
@jerch
Copy link
Member Author

jerch commented Oct 24, 2019

@Tyriar To not get stale I removed DECLRMM from this PR as its more involved and better done separately.

Up for review.

@jerch
Copy link
Member Author

jerch commented Oct 24, 2019

Geez not yet done with this PR - DECIC and DECDC are margin aware, thus should only shift content between margins, its a NOOP outside. See http://www.bitsavers.org/pdf/dec/standards/EL-SM070-00_DEC_STD_070_Video_Systems_Reference_Manual_Dec91.pdf#page=406&zoom=auto,-38,762
Also for SL/SR xterm seems to apply margins...

Copy link
Member

@Tyriar Tyriar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do any apps of note use this one that you're aware of?

@jerch
Copy link
Member Author

jerch commented Oct 24, 2019

@Tyriar With DECLRMM it can be used by apps to do partial viewport updates (like splitting in a multiplexer and such). Since this is supported for long in xterm and other emulators apps might shift to it, not sure if any does atm. Maybe ncurses already uses this, idk.

Already fixed in with the last commit, tests still to come.

@Tyriar
Copy link
Member

Tyriar commented Oct 24, 2019

@jerch k, ping me when ready to review 😃

@jerch
Copy link
Member Author

jerch commented Oct 24, 2019

@Tyriar Ping 😄

@Tyriar Tyriar added this to the 4.2.0 milestone Oct 24, 2019
@jerch jerch merged commit 527379d into xtermjs:master Oct 24, 2019
@jerch jerch deleted the left_right_scroll branch October 24, 2019 20:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants