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.

Sign up for GitHub

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

Jump to bottom

Wide characters are only half underlined #158981

Closed
TylerLeonhardt-alt opened this issue Aug 23, 2022 · 1 comment · Fixed by #159494
Closed

Wide characters are only half underlined #158981

TylerLeonhardt-alt opened this issue Aug 23, 2022 · 1 comment · Fixed by #159494
Assignees
@Tyriar
Labels
bug Issue identified by VS Code Team member as probable bug terminal-rendering upstream Issue identified as 'upstream' component related (exists outside of VS Code) upstream-issue-fixed The underlying upstream issue has been fixed upstream-issue-linked This is an upstream issue that has been reported upstream verified Verification succeeded
Milestone
September 2022

Comments

@TylerLeonhardt-alt
Copy link

Testing #158827

👽 現代

image

@TylerLeonhardt 's alt
@Tyriar Tyriar changed the title complex characters are only half underlined Wide characters are only half underlined Aug 23, 2022
@Tyriar Tyriar added bug Issue identified by VS Code Team member as probable bug terminal-rendering labels Aug 23, 2022
@Tyriar Tyriar added this to the Backlog milestone Aug 23, 2022
@Tyriar Tyriar mentioned this issue Aug 23, 2022
Closed
@Tyriar
Copy link
Member

Tyriar commented Aug 23, 2022

Upstream: xtermjs/xterm.js#4063

@Tyriar Tyriar added upstream Issue identified as 'upstream' component related (exists outside of VS Code) upstream-issue-linked This is an upstream issue that has been reported upstream upstream-issue-fixed The underlying upstream issue has been fixed labels Aug 23, 2022
@Tyriar Tyriar modified the milestones: Backlog, September 2022 Aug 29, 2022
Tyriar added a commit that referenced this issue Aug 29, 2022
@Tyriar
Update xterm.js
b02e85c 
Fixes #159492
Fixes #159493
Fixes #158868
Fixes #158981
Fixes #157432
@Tyriar Tyriar mentioned this issue Aug 29, 2022
Merged
@roblourens roblourens added the verified Verification succeeded label Sep 29, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Oct 13, 2022
lemanschik pushed a commit to code-oss-dev/code that referenced this issue Nov 25, 2022
@Tyriar
Update xterm.js
77eeef2 
Fixes microsoft#159492
Fixes microsoft#159493
Fixes microsoft#158868
Fixes microsoft#158981
Fixes microsoft#157432
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Tyriar Tyriar

Labels
bug Issue identified by VS Code Team member as probable bug terminal-rendering upstream Issue identified as 'upstream' component related (exists outside of VS Code) upstream-issue-fixed The underlying upstream issue has been fixed upstream-issue-linked This is an upstream issue that has been reported upstream verified Verification succeeded
Projects
None yet
Milestone
September 2022
Development

Successfully merging a pull request may close this issue.

Update xterm.js microsoft/vscode
4 participants
@roblourens @Tyriar @meganrogge @TylerLeonhardt-alt