-
Notifications
You must be signed in to change notification settings - Fork 9.1k
Description
Environment
Windows build number: Version 10.0.18362.657
Windows Terminal version (if applicable): Commit 7ae3433
Steps to reproduce
- Build commit 7ae3433
- Open a bash shell
- Execute
printf "\u2660\u2663\u2665\u2666\n"
Expected behavior
This should print our the four card suite Unicode characters, each being one cell wide. Here's what it used to look like in conhost:
Actual behavior
Each characters occupies two cells now.
I wouldn't be too concerned if these were some recently invented emoji like crying kitten, or dancing monkey with umbrella. But these characters are from the default cmd shell code page (CP437), and they've been narrow characters for going on 40 years now. They're even included in the WGL4 character repertoire. Changing their width is almost guaranteed to break things.
I'm sorry I didn't bring this up before PR #5795 was merged, but I wasn't paying attention to the actual characters that were being changed. I only noticed now when I was testing some of my own code and found that this broke Vttest (it uses the diamond character in a couple of the tests).

