New Rule: require-button-type

It is very easy to forget type="button" on <button> elements and accidentally create a submit button. I think a rule to enforce the usage of an explicit type attribute on buttons might make sense.

@rwjblue thoughts?

[Turbo87]

not sure, I think it probably should still set an explicit default value.

[johanrd]

OK, I understand.

A sensible approach in my case could then be to disable the global require-button-type, and instead add an instruction manually before every relevant form tag:

{{!-- template-lint-enable require-button-type --}}
<form>

…I guess it is a matter of taste whether require-button-type should be true or false in the default config. Since the rule goes beyond its intended scope, I’d lean towards defaulting it to false, but I guess it may also be a good trade-off for most people to just write type="button" on all buttons.