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

Should markdown url smart paste be disabled if you're pasting onto an existing url? #188872

Closed
Tracked by #189320
mjbvz opened this issue Jul 25, 2023 · 0 comments · Fixed by #189347
Closed
Tracked by #189320

Should markdown url smart paste be disabled if you're pasting onto an existing url? #188872

mjbvz opened this issue Jul 25, 2023 · 0 comments · Fixed by #189347
Assignees
Labels
insiders-released Patch has been released in VS Code Insiders
Milestone

Comments

@mjbvz
Copy link
Collaborator

mjbvz commented Jul 25, 2023

  1. In a md file:
http://example.com/1

http://example.com/2
  1. Copy the first line http://example.com/1
  2. Select all of http://example.com/2 and paste

Bug
This creates:

[http://example.com/2](http://example.com/1)

It seems unlikely that users would want this even though it is the consistent behavior

@MeghanKulkarni MeghanKulkarni added this to the August 2023 milestone Jul 31, 2023
@beto811 beto811 mentioned this issue Jul 31, 2023
@vscodenpa vscodenpa 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 Jul 31, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Sep 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
insiders-released Patch has been released in VS Code Insiders
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants