Add wide character rendering repro app for terminal bug reports#5397
Conversation
Minimal standalone .NET console app that demonstrates incorrect wide/fullwidth Unicode character rendering in terminal emulators. Outputs grids of emoji, CJK, and mixed-width characters with box-drawing separators that should align vertically. If they don't, the terminal has a wcwidth bug. Intended for use in bug reports against terminals that mishandle wide chars. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 178ee955d8
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Add Ghostty to the terminal compatibility matrix in BUG_REPORT_TEMPLATE.md as passing all tests. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
0c576be to
d118b6c
Compare
…ails Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
I haven't had the opportunity to review this PR. I'm just asking if the use of ZWJ emojis was carefully considered, as it's also considered a wide character occupying two columns? |
sorry, but I needed to merge to move on and since its' just as sample... I just used codepoints that demonstrated the Github Copilot App terminal bug: If you think this test app can be improved, that'd be awesome! |
Fixes
Proposed Changes/Todos
Adds a minimal standalone console app (
Examples/WideCharRepro/) that demonstrates incorrect wide/fullwidth Unicode character rendering in terminal emulators. This is intended as a repro to attach when filing bugs against terminals that mishandle wide characters (emoji, CJK, fullwidth forms).The app outputs 5 tests:
If the
|separators are not vertically aligned, the terminal has awcwidthbug.Also includes:
README.md-- explains the problem, how to run, and relevant Unicode standardsBUG_REPORT_TEMPLATE.md-- copy-paste template for filing issues against terminals, with a compatibility matrixPull Request checklist:
dotnet testbefore commit///style comments)