width, align, valign, cellpadding, cellspacing, border, summary, bgcolor
Error: The width attribute on the table element is obsolete. Use CSS instead.
Error: The align attribute on the table element is obsolete. Use CSS instead.
Error: The valign attribute on the table element is obsolete. Use CSS instead.
Error: The cellpadding attribute on the table element is obsolete. Use CSS instead.
Error: The cellspacing attribute on the table element is obsolete. Use CSS instead.
Error: The border attribute on the table element is obsolete. Use CSS instead.
Error: The summary attribute on the table element is obsolete. Consider describing the structure of the table in a caption element or in a figure element containing the table; or, simplify the structure of the table so that no description is needed.
Error: The bgcolor attribute on the table element is obsolete. Use CSS instead.
Error: The element button must not appear as a descendant of the a element.
Specification:
http://w3c.github.io/html/textlevel-semantics.html#the-a-element
"Allowed ARIA role attribute values: link (default - do not set), button, checkbox, radio, switch, tab or treeitem"
"The <a> element may be wrapped around entire paragraphs, lists, tables, and so forth, even entire sections, so long as there is no interactive content within (e.g., buttons or other links)."
There is something strange here, the $force_reload is unused and
after stop we don't check the service status so 'running' is still
true and it tries to reload instead.
Best time to refactor into a mutable service controller?!
PR: https://github.com/opnsense/plugins/issues/454
CC: @fraenki
NOYB
Franco Fichtner
Michael
Fabian Franz, BSc
Frank Wall