diff options
author | patrickhlauke <redux@splintered.co.uk> | 2017-10-31 16:50:31 +0300 |
---|---|---|
committer | XhmikosR <xhmikosr@gmail.com> | 2017-10-31 22:36:17 +0300 |
commit | ce90b82b19d08673ac529c72e3fe5e6a1c0b607b (patch) | |
tree | 189880454a806c4570012bc1a1924034c3130fe8 /build | |
parent | 95025a992fdab7234b0f6b27aa61c2d848288816 (diff) |
Add comments to clarify why we ignore a couple of vnu's errors.
* "autocomplete" attribute errors
* "legend" is not allowed as child of "div"
Diffstat (limited to 'build')
-rw-r--r-- | build/vnu-jar.js | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/build/vnu-jar.js b/build/vnu-jar.js index ea894ac06c..7ac6ed5472 100644 --- a/build/vnu-jar.js +++ b/build/vnu-jar.js @@ -22,12 +22,16 @@ childProcess.exec('java -version', (error, stdout, stderr) => { // vnu-jar accepts multiple ignores joined with a `|` const ignores = [ + // "autocomplete" is included in <button> and checkboxes and radio <input>s due to + // Firefox's non-standard autocomplete behavior - see https://bugzilla.mozilla.org/show_bug.cgi?id=654072 'Attribute “autocomplete” is only allowed when the input type is “color”, “date”, “datetime-local”, “email”, “hidden”, “month”, “number”, “password”, “range”, “search”, “tel”, “text”, “time”, “url”, or “week”.', 'Attribute “autocomplete” not allowed on element “button” at this point.', 'Attribute “title” not allowed on element “circle” at this point.', 'Bad value “tablist” for attribute “role” on element “nav”.', // We use holder.js with `data-src` and no `src`; we could work around this, not sure it's worth it. 'Element “img” is missing required attribute “src”.', + // Markup used in Components > Forms > Layout > Form grid > Horizontal form is currently invalid, + // but used this way due to lack of support for flexbox layout on <fieldset> element in most browsers 'Element “legend” not allowed as child of element “div” in this context.*', 'The “datetime-local” input type is not supported in all browsers. Please be sure to test, and consider using a polyfill.', // The next one we are using it because IE11 doesn't recognise <main>. |