remove redundant for/id attributes from wrapped <label> tags
The <input elements are are wrapped <label>, so the association is clear and no for/id mapping is required.
use
instead of as wrapper around settings
For valid HTML the <fieldset> should have a <legend> element. We might move the headline (<h3>) into the legend, but actually we only have one set of fields in the form, so additional semantic grouping is not necessary.
do not close tags
We close the tags for number inputs, but not for checkboxes. Let's use the same way for all inputs.
add type attribute to refresh button
Type attribute is required. All browsers tested seem to assume "submit" by default (otherwise the mechanism would not work), but we should make it explicit.
<label>
tags The<input
elements are are wrapped<label>
, so the association is clear and no for/id mapping is required.<fieldset>
should have a<legend>
element. We might move the headline (<h3>
) into the legend, but actually we only have one set of fields in the form, so additional semantic grouping is not necessary.Kudos, SonarCloud Quality Gate passed!
0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells
No Coverage information
No Duplication information