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

Invalid handling emoji with zero-width-space and \u200b zero-width-space character. #10495

Closed
zagura opened this issue Jun 22, 2021 · 2 comments
Labels
Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing.

Comments

@zagura
Copy link

zagura commented Jun 22, 2021

Windows Terminal version (or Windows build number)

10.0.18363.1556, 1.8.1521.0

Other Software

OpenSSH_7.6p1 Ubuntu-4ubuntu0.3, (inside WSL)

Steps to reproduce

  1. echo Some$'\u200b'Text
  2. echo "🖼"and https://example.com
    Second symbol is framed-picture emoji: https://emojipedia.org/framed-picture/

Expected Behavior

  1. Some​Text (without visible space)
  2. No additional space after emoji and link https://example.com is clickable, so Ctrl + LPM would open https://example.com URL.

Actual Behavior

  1. Some Text (encoded correctly, but displayed space between words)
  2. Displayed text has additional space between emoji and "and" word. When mouse hoover, underline is moved one character to right, so here it would underline ttps://example.com (and empty character after URL). Ctrl + LPM would try to open invalid URL ttps://example.com.
@ghost ghost added Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Tag-Fix Doesn't match tag requirements labels Jun 22, 2021
@DHowett
Copy link
Member

DHowett commented Jun 22, 2021

Thanks! This is a huge class of issue that we're tracking in /dupe #1472 and #3546 😄

@ghost
Copy link

ghost commented Jun 22, 2021

Hi! We've identified this issue as a duplicate of another one that already exists on this Issue Tracker. This specific instance is being closed in favor of tracking the concern over on the referenced thread. Thanks for your report!

@ghost ghost closed this as completed Jun 22, 2021
@ghost ghost added Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing. and removed Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Tag-Fix Doesn't match tag requirements labels Jun 22, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing.
Projects
None yet
Development

No branches or pull requests

2 participants