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

Dropping multiple files into markdown inserts stray %0D #175547

Closed
mjbvz opened this issue Feb 27, 2023 · 0 comments · Fixed by #175552
Closed

Dropping multiple files into markdown inserts stray %0D #175547

mjbvz opened this issue Feb 27, 2023 · 0 comments · Fixed by #175552
Assignees
@mjbvz
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders markdown Markdown support issues verified Verification succeeded
Milestone
March 2023

Comments

@mjbvz
Copy link
Contributor

mjbvz commented Feb 27, 2023 •
edited

Type: Bug

  1. Drop multiple files from the VS Code explorer into a markdown file

Bug
An extra %0D is inserted:

[label](file1.md%0D) [label](file2.md)

VS Code version: Code - Insiders 1.76.0-insider (Universal) (ead95b4, 2023-02-24T22:01:16.601Z)
OS version: Darwin x64 22.3.0
Modes:
Sandboxed: Yes

System Info
Item Value
CPUs Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz (16 x 2400)
GPU Status 2d_canvas: enabled
canvas_oop_rasterization: disabled_off
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
metal: disabled_off
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: enabled
raw_draw: disabled_off_ok
skia_renderer: enabled_on
video_decode: enabled
video_encode: enabled
vulkan: disabled_off
webgl: enabled
webgl2: enabled
webgpu: disabled_off
Load (avg) 4, 5, 4
Memory (System) 32.00GB (0.23GB free)
Process Argv --crash-reporter-id 48781ca2-1705-4f64-9bab-325055aab55d
Screen Reader no
VM 0%
@mjbvz mjbvz self-assigned this Feb 27, 2023
@mjbvz mjbvz added bug Issue identified by VS Code Team member as probable bug markdown Markdown support issues labels Feb 27, 2023
@mjbvz mjbvz added this to the March 2023 milestone Feb 27, 2023
mjbvz added a commit to mjbvz/vscode that referenced this issue Feb 27, 2023
@mjbvz
Split uri-lists on \r\n for markdown drop
7d73198 
Fixes microsoft#175547
@mjbvz mjbvz mentioned this issue Feb 27, 2023
Merged
mjbvz added a commit that referenced this issue Feb 27, 2023
@mjbvz
Split uri-lists on \r\n for markdown drop (#175552)
7570fad 
Fixes #175547
@VSCodeTriageBot VSCodeTriageBot added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Feb 27, 2023
@meganrogge meganrogge added the verified Verification succeeded label Mar 22, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Apr 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mjbvz mjbvz

Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders markdown Markdown support issues verified Verification succeeded
Projects
None yet
Milestone
March 2023
Development

Successfully merging a pull request may close this issue.

Split uri-lists on \r\n for markdown drop mjbvz/vscode
3 participants
@mjbvz @meganrogge @VSCodeTriageBot