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

Adds the support of the formaction attribute #36

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kakone
Copy link

@kakone kakone commented Apr 24, 2018

An HTML form can have multiple submit buttons with differents actions. In this case, the HTML5 formaction attribute overrides the action attribute of the form element.

An HTML form can have multiple submit buttons with differents actions. In this case, the [HTML5 formaction attribute](https://www.w3schools.com/tags/att_input_formaction.asp) overrides the action attribute of the form element.
@kakone
Copy link
Author

kakone commented Jul 25, 2018

@Eilon It would be nice if someone can review and merge this little pull request

@Eilon
Copy link
Member

Eilon commented Jul 25, 2018

@kakone it's best if there is an issue tracking this because we review issues initially to make determinations. If you can log an issue and briefly describe the scenario, we will review it, and then look at the PR as well. Thanks!

@nprieto95
Copy link

@Eilon I have opened an Issue, #40 , in which I briefly describe a real world scenario where I have found that this solution could be really helpful. I strongly believe it would be a good enhancement to the product.

@Eilon
Copy link
Member

Eilon commented Jul 26, 2018

@nprieto95 thanks, we will review the issue and go from there (BTW the person who would look at this issue is on vacation right now so it might be a few days before we get to it).

@mkArtakMSFT
Copy link
Member

Closing this PR as the associated issue is closed.

@mkArtakMSFT
Copy link
Member

Reopening the PR, as the associated Issue is now reopened.

@Gustavpaeklund
Copy link

I would like to see this feature merged from Microsoft.

@kakone
Copy link
Author

kakone commented Feb 7, 2020

I now also added the support of the formmethod attribute.

Base automatically changed from master to main March 15, 2021 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants