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

Fix #120 Add reportLayerColor, getReportedLayerColor, getLayerColor #137

Merged
merged 2 commits into from
Nov 21, 2022

Conversation

mpilgrem
Copy link
Collaborator

@mpilgrem mpilgrem commented May 4, 2022

Also adds hReportLayerColor, hGetLayerColor and layerColor.

Not supported by terminals on Windows. Native terminals do not allow the color to be reported. Other terminals do not allow the reported color to be read from the console input stream.

@Bodigrim
Copy link
Contributor

Is this related to UnkindPartition/tasty#298?

@mpilgrem
Copy link
Collaborator Author

@Bodigrim, I had not seen that. This PR has been on the back burner, partly because the feature is not really cross-platform yet, even with emulation, for the reasons discussed in the related issue #120. There also does not seem to have been much progress on the Windows Terminal issue. However, I'll think about moving it into the code base.

…getLayerColor

Also adds `hReportLayerColor`, `hGetLayerColor` and `layerColor`.

On Windows, emulated using the Windows Console API but the emulation of `reportLayerColor` works only on the native ConHost terminal and not on Windows Terminal. Non-native terminals cannot make use of `getReportedLayerColor`.
@mpilgrem mpilgrem merged commit 1ca977b into UnkindPartition:master Nov 21, 2022
@mpilgrem mpilgrem deleted the fix120 branch November 21, 2022 22:43
@Bodigrim
Copy link
Contributor

Great stuff, thanks for release!

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