Skip to content

Conversation

@Ducki
Copy link
Contributor

@Ducki Ducki commented Oct 12, 2022

The <dialog> element has also introduced a new form method dialog, see https://html.spec.whatwg.org/multipage/form-control-infrastructure.html#attr-fs-formmethod

This PR adds the Dialog field to the FormMethod enum.

Fixes #44499

The <dialog> element has also introduced a new form method `dialog`, see https://html.spec.whatwg.org/multipage/form-control-infrastructure.html#attr-fs-formmethod

This PR adds the `Dialog` field to the FormMethod enum.
@Ducki Ducki requested a review from a team as a code owner October 12, 2022 16:46
@ghost ghost added area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates community-contribution Indicates that the PR has been added by a community member labels Oct 12, 2022
@ghost
Copy link

ghost commented Oct 12, 2022

Thanks for your PR, @Ducki. Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

@mkArtakMSFT
Copy link
Contributor

@javiercn can you please review this? Thanks!

@javiercn javiercn merged commit 34cd823 into dotnet:main Oct 18, 2022
@ghost ghost added this to the 8.0-preview1 milestone Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for new dialog form method in FormMethod enum

3 participants