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

Default Form Method is misleading #4

Closed
benschiefer opened this issue Sep 14, 2018 · 0 comments
Closed

Default Form Method is misleading #4

benschiefer opened this issue Sep 14, 2018 · 0 comments

Comments

@benschiefer
Copy link

When dragging a form onto the canvas, it does not set the "method" attribute in the markup (which is fine). However, the default "method" for a form is "GET" according to the HTML spec.

If you select the form block that you dragged onto the canvas and look at the component settings, the UI makes it look like the user has the "POST" method selected, since it is the first element in the select.

Recommendation: make the "GET" option the first element in the select so that it matches the default form behavior.

https://github.com/artf/grapesjs-plugin-forms/blob/master/src/components.js#L76

@artf artf closed this as completed in 4d59c39 Dec 21, 2018
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

No branches or pull requests

1 participant