Skip to content

Conversation

KronosTheLate
Copy link
Contributor

Replaces #50154.

@giordano
Copy link
Member

Can you please use a more descriptive title? This doesn't convey any useful information.

@LilithHafner LilithHafner changed the title Re-implement original PR Set default TextTerminal, add csave and crestore Oct 11, 2025
@KronosTheLate
Copy link
Contributor Author

KronosTheLate commented Oct 12, 2025

Thanks @LilithHafner

Also, the code changes are small and all in one file, so just having a look at the changes is quite fast and easy for this PR in particular.

@LilithHafner
Copy link
Member

It would also be good to put some of the justification for these API expansions into the OP, so folks know why to consider this PR.

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.

4 participants