Skip to content

Conversation

@rgetz
Copy link
Contributor

@rgetz rgetz commented Nov 28, 2025

generated html reports were failing Nu Html Checker at https://validator.w3.org/

  • Add space before checkbox attributes to comply with HTML5 standards.
  • Remove trailing slashes from elements to prevent parser warnings.
  • Correct colspan in summary and fileEntry tables from 5 to 6 to match header column count, preventing table validation warnings.

generated html reports were failing Nu Html Checker at https://validator.w3.org/

- Add space before checkbox attributes to comply with HTML5 standards.
- Remove trailing slashes from elements to prevent parser warnings.
- Correct colspan in summary and fileEntry tables from 5 to 6 to match header
  column count, preventing table validation warnings.

Signed-off-by: Robin Getz <[email protected]>
@sonarqubecloud
Copy link

@firewave
Copy link
Collaborator

There currently is a HTML validation step using tidy in the CI. If you are aware of a different tool which is equivalent to the online validator feel free to mention it. It might also be possible that the current check is just misconfigured.

@rgetz
Copy link
Contributor Author

rgetz commented Nov 29, 2025

While Tidy passes without errors or warnings, other validators (like Nu Html Checker) report issues. This is expected because:

  • Tidy is more lenient
  • Tidy primarily checks for well-formedness and common mistakes, and ignores some strict HTML5 rules.
  • It allows some outdated syntax or minor structural issues (e.g., unquoted attributes, void element slashes) that modern validators consider errors.

Other validators are stricter. Tools like the Nu Html Checker enforce HTML5 conformance rigorously, including:

  • Proper void element syntax ( not )
  • Attribute quoting
  • Table column consistency
  • Inline event handler conventions

These stricter rules may generate warnings even though the tidy HTML “works” in browsers...

Check out:

@firewave
Copy link
Collaborator

While Tidy passes without errors or warnings, other validators (like Nu Html Checker) report issues.

Thanks. I filed https://trac.cppcheck.net/ticket/14297 about replacing tidy.

@danmar danmar merged commit f65e41b into danmar:main Dec 7, 2025
55 checks passed
@danmar
Copy link
Owner

danmar commented Dec 7, 2025

@rgetz I would like to add your name in the AUTHORS file. which name would you like that I add?

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.

3 participants