Skip to content

Conversation

@sgraband
Copy link
Contributor

What it does

Added the optional enableForms field to WebviewOptions.
This allows the allow-forms sandbox attribute to be set.

Resolves #11517.

Contributed on behalf of STMicroelectronics

How to test

To test you can use the following VSCode extension:

webview-forms-0.0.1.zip

It contributes 6 commands (category: Test enableForms) that will create a webview with different options set (You can see the options set in the command name and the webview will display them as well).
When you opened the webview you can inspect the DevTools to see that the allow-forms sandbox attribute is correctly set (or not set).

Review checklist

Reminder for reviewers

Added the optional `enableForms` field to `WebviewOptions`.
This allows the `allow-forms` sandbox attribute to be set.

Contributed on behalf of STMicroelectronics
Co-authored-by: Vincent Fugnitto <[email protected]>
@vince-fugnitto vince-fugnitto added vscode issues related to VSCode compatibility webviews issues related to webviews labels Dec 14, 2022
Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍 I confirmed that the changes work as expected, and the API is marked as supported.

@sgraband
Copy link
Contributor Author

Thank you for the review @vince-fugnitto! Can we merge this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

vscode issues related to VSCode compatibility webviews issues related to webviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[vscode] Support optional WebviewOptions.enableForms

3 participants