Skip to content

Fix Markdown Preview incorrectly encodes a link#61530

Merged
mjbvz merged 1 commit intomicrosoft:masterfrom
sankt-petersbug:fix-60525
Oct 22, 2018
Merged

Fix Markdown Preview incorrectly encodes a link#61530
mjbvz merged 1 commit intomicrosoft:masterfrom
sankt-petersbug:fix-60525

Conversation

@sankt-petersbug
Copy link
Contributor

For known external scheme, the customized normalizeLink method use vscode.Uri to parse and create
the string representation of the link before caling the original normalizeLink method.
The toString method of vscode.Uri encodes the result by default, and this is unecessary since encoding
is handled by the original normalizeLink method.

Calls toString method with skipEncoding option.

Resolves: #60525

For known external scheme, the customized normalizeLink method use vscode.Uri to parse and create
the string representation of the link before caling the original normalizeLink method.
The toString method of vscode.Uri encodes the result by default, and this is unecessary since encoding
is handled by the original normalizeLink method.

Calls toString method with skipEncoding option.

Resolves: microsoft#60525
@mjbvz mjbvz added this to the October 2018 milestone Oct 22, 2018
@mjbvz mjbvz merged commit f2653c0 into microsoft:master Oct 22, 2018
@mjbvz
Copy link
Collaborator

mjbvz commented Oct 22, 2018

Thanks!

@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Markdown preview urlencode link get arguments

2 participants