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.

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

Make cursor save/restore affect color attributes #330

Merged
merged 2 commits into from
Jun 5, 2017

Conversation

whydoubt
Copy link
Contributor

@whydoubt whydoubt commented Jun 2, 2017

SGR attributes are stored in three variables: mEffect, mForeColor, and
mBackColor. Saving/restoring the cursor only preserves mEffect.

Change the cursor save/restore methods to additionally preserve
mForeColor and mBackColor. This affects both 'explit' saving/restoring
the cursor and switching to/from the alternate screen buffer.

whydoubt added 2 commits June 3, 2017 17:45
SGR attributes are stored in three variables: mEffect, mForeColor, and
mBackColor.  Saving/restoring the cursor only preserves mEffect.

Change the cursor save/restore methods to additionally preserve
mForeColor and mBackColor.  This affects both 'explit' saving/restoring
the cursor and switching to/from the alternate screen buffer.
@whydoubt whydoubt force-pushed the cursor_preserve_color branch from c7bcaa6 to 2e93837 Compare June 4, 2017 00:19
@fornwall fornwall merged commit c06770c into termux:master Jun 5, 2017
@fornwall
Copy link
Member

fornwall commented Jun 5, 2017

Thanks a lot - this is included in the v0.51 update that is now rolling out as an update!

AdamMickiewich pushed a commit to VolyaTeam/dzida-app that referenced this pull request Aug 8, 2022
Make cursor save/restore affect color attributes
shrihankp pushed a commit to reisxd/termux-app that referenced this pull request Oct 20, 2022
Make cursor save/restore affect color attributes
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