microsoft / vscode

Visual Studio Code
https://code.visualstudio.com
MIT License
163.76k stars 29.1k forks source link

Something wrong with URIs/tabs on Windows #64213

Closed Tyriar closed 5 years ago

Tyriar commented 5 years ago

VS Code version: Code - Insiders 1.30.0-insider (9e62a05bc7c5dbb74d165e06f7c3a5f3c7549096, 2018-12-03T06:19:31.620Z) OS version: Windows_NT x64 10.0.18290

System Info |Item|Value| |---|---| |CPUs|Intel(R) Xeon(R) CPU E5-1620 v4 @ 3.50GHz (8 x 3492)| |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: disabled_software
rasterization: unavailable_software
video_decode: enabled
video_encode: enabled
webgl: enabled
webgl2: enabled| |Memory (System)|63.92GB (52.72GB free)| |Process Argv|| |Screen Reader|no| |VM|0%|
Extensions (12) Extension|Author (truncated)|Version ---|---|--- vscode-svgviewer|css|1.4.7 EditorConfig|Edi|0.12.5 tslint|eg2|1.0.40 git-project-manager|fel|1.7.1 vscode-pull-request-github|Git|0.3.0 vscode-scss|mrm|0.6.2 PowerShell|ms-|1.9.0 material-icon-theme|PKi|3.6.1 seti-icons|qin|0.1.3 lorem-ipsum|Tyr|1.2.0 vscode-todo-highlight|way|1.0.4 vscode-open-in-github|ziy|1.3.6 (1 theme extensions excluded)

Notice the breadcrumbs:

image

Make a change to the file:

image

Switch to the other file:

image

I originally created https://github.com/Microsoft/vscode/issues/63807 but it turns out it's not a problem with search.

I've been seeing this on and off for the past week in multiple workspaces. The only interesting thing I can think of is my Insiders went ~3 weeks without updating.

bpasero commented 5 years ago

@jrieken sounds like the URI migration issue (for the drive letter) to me, but that one happened last release already right?

bpasero commented 5 years ago

My guess is that this is just an unlucky case where the tab had the old format and was never closed and reopened ever since.

jrieken commented 5 years ago

sounds like the URI migration issue (for the drive letter) to me, but that one happened last release already right?

We reverted all of that again, it was in insiders for a two or three weeks only. All of that happened during the previous milestone, not during 1.30-development

bpasero commented 5 years ago

Ok.