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

Terminal link underlines on lines that start with \t are offset #66632

Closed
ejizba opened this issue Jan 16, 2019 · 2 comments
Closed

Terminal link underlines on lines that start with \t are offset #66632

ejizba opened this issue Jan 16, 2019 · 2 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug terminal General terminal issues that don't fall under another label upstream Issue identified as 'upstream' component related (exists outside of VS Code) verified Verification succeeded
Milestone

Comments

@ejizba
Copy link

ejizba commented Jan 16, 2019

Issue Type: Bug

I've started noticing the line under the hyperlink looks weird on insiders:
screen shot 2019-01-16 at 2 45 38 pm

This is what it looks like on non-insiders:
screen shot 2019-01-16 at 2 46 05 pm

I think I narrowed it down to when there's a tab and a link. Here's the simplest task I could create that repros this:

{
  "version": "2.0.0",
  "tasks": [
    {
      "label": "test",
      "type": "shell",
      "command": "echo '	http://localhost:7071/api/HttpTrigger'",
      "problemMatcher": []
    }
  ]
}

VS Code version: Code - Insiders 1.31.0-insider (b4fc71b, 2019-01-15T06:12:40.322Z)
OS version: Darwin x64 18.2.0

System Info
Item Value
CPUs Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz (8 x 2500)
GPU Status 2d_canvas: enabled
checker_imaging: disabled_off
flash_3d: enabled
flash_stage3d: enabled
flash_stage3d_baseline: enabled
gpu_compositing: enabled
multiple_raster_threads: enabled_on
native_gpu_memory_buffers: enabled
rasterization: enabled
surface_synchronization: enabled_on
video_decode: enabled
webgl: enabled
webgl2: enabled
Load (avg) 2, 2, 2
Memory (System) 16.00GB (0.57GB free)
Process Argv
Screen Reader no
VM 0%
Extensions (6)
Extension Author (truncated) Version
tslint eg2 1.0.42
python ms- 2018.12.1
azure-account ms- 0.8.0
csharp ms- 1.17.1
java red 0.36.0
vscode-java-debug vsc 0.16.0
@vscodebot vscodebot bot added the terminal General terminal issues that don't fall under another label label Jan 16, 2019
@Tyriar Tyriar added this to the December/January 2019 milestone Jan 17, 2019
@Tyriar Tyriar added bug Issue identified by VS Code Team member as probable bug upstream Issue identified as 'upstream' component related (exists outside of VS Code) labels Jan 17, 2019
@Tyriar
Copy link
Member

Tyriar commented Jan 31, 2019

Upstream PR: xtermjs/xterm.js#1887, too late to pull into 1.31

@Tyriar Tyriar modified the milestones: February 2019, March 2019 Feb 23, 2019
@Tyriar Tyriar changed the title Line under hyperlink rendered improperly in terminal Terminal link underlines on lines that start with \t are offset Mar 1, 2019
@Tyriar Tyriar closed this as completed in 1c41ee3 Mar 8, 2019
@ejizba
Copy link
Author

ejizba commented Mar 11, 2019

Fixed for me on latest insiders - thanks!

@Tyriar Tyriar added the verified Verification succeeded label Mar 11, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Apr 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug terminal General terminal issues that don't fall under another label upstream Issue identified as 'upstream' component related (exists outside of VS Code) verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

2 participants