Skip to content

fix download issues in webviews#108603

Merged
mjbvz merged 2 commits intomicrosoft:masterfrom
pi-r-p:fixWebViewDownload
Oct 14, 2020
Merged

fix download issues in webviews#108603
mjbvz merged 2 commits intomicrosoft:masterfrom
pi-r-p:fixWebViewDownload

Conversation

@pi-r-p
Copy link
Contributor

@pi-r-p pi-r-p commented Oct 13, 2020

This PR fixes #108602

@pi-r-p
Copy link
Contributor Author

pi-r-p commented Oct 13, 2020

I am pretty sure other Visual Studio Code extension developers will get the same problem...

@mjbvz mjbvz added this to the October 2020 milestone Oct 13, 2020
@mjbvz
Copy link
Collaborator

mjbvz commented Oct 13, 2020

Make sure to also add this in

element.sandbox.add('allow-scripts', 'allow-same-origin', 'allow-forms', 'allow-pointer-lock');

@ghost
Copy link

ghost commented Oct 14, 2020

CLA assistant check
All CLA requirements met.

@pi-r-p
Copy link
Contributor Author

pi-r-p commented Oct 14, 2020

Done. I was wondering why there is typescript + javascript here. Modifying typescript only do not have effect on the final build. Is there a transition ongoing ?

@mjbvz mjbvz merged commit 74335e9 into microsoft:master Oct 14, 2020
@mjbvz
Copy link
Collaborator

mjbvz commented Oct 14, 2020

Thanks!

jmannanc pushed a commit to jmannanc/vscode that referenced this pull request Oct 26, 2020
* fix download issues in webviews

* also add allow-download in ts src
@github-actions github-actions bot locked and limited conversation to collaborators Dec 4, 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.

Unable to click download button in a webview

2 participants