Skip to content
This repository was archived by the owner on Aug 6, 2023. It is now read-only.

Support more style modifiers on Windows and fix build with Crossterm 0.17.8 #368

Merged
merged 2 commits into from
Sep 20, 2020

Conversation

alvinhochun
Copy link
Contributor

@alvinhochun alvinhochun commented Sep 7, 2020

Fixes #373.

Crossterm might actually do WinAPI calls instead of writing ANSI excape
codes so writing to an intermediate String may cause issues on older
versions of Windows. It also fails to compile with Crossterm 0.17.8 due
to Crossterm now expecting the writer to support `flush`, which String
doesn't.

Fixes fdehau#373
@alvinhochun alvinhochun changed the title Support more style modifiers on Windows Support more style modifiers on Windows and fix build with Crossterm 0.17.8 Sep 10, 2020
@fdehau fdehau merged commit e0b2572 into fdehau:master Sep 20, 2020
@fdehau
Copy link
Owner

fdehau commented Sep 20, 2020

Thanks for taking the time to look into this :)

TheLostLambda pushed a commit to TheLostLambda/tui-rs that referenced this pull request Sep 24, 2020
…and fix build with Crossterm 0.17.8 (fdehau#368)

* Support more style modifiers on Windows
* Change Crossterm backend to write directly to buffer instead of String

Crossterm might actually do WinAPI calls instead of writing ANSI excape
codes so writing to an intermediate String may cause issues on older
versions of Windows. It also fails to compile with Crossterm 0.17.8 due
to Crossterm now expecting the writer to support `flush`, which String
doesn't.

Fixes fdehau#373
24seconds pushed a commit to 24seconds/tui-rs that referenced this pull request Nov 30, 2020
…and fix build with Crossterm 0.17.8 (fdehau#368)

* Support more style modifiers on Windows
* Change Crossterm backend to write directly to buffer instead of String

Crossterm might actually do WinAPI calls instead of writing ANSI excape
codes so writing to an intermediate String may cause issues on older
versions of Windows. It also fails to compile with Crossterm 0.17.8 due
to Crossterm now expecting the writer to support `flush`, which String
doesn't.

Fixes fdehau#373
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Macro error when compiling
2 participants