The `role` attribute is not allowed by the XHTML 1.0 Strict DTD, so we
should only emit it in HTML5 mode (and we should probably
-[[todo/generate_HTML5_by_default]]).
+[[todo/generate_HTML5_by_default]]). --[[smcv]]
+
+> Now we do. --[[smcv]]
+
+Specific roles:
[[!format diff """
-<div class="banner">
in HTML5 mode is enough to give accessibility tools a clue already?
Would declaring this to be a `note` be sufficient?
+I've applied your suggested roles for #main, #comments and #footer,
+but only in HTML5 mode for the reason given above. I have not applied
+a role to the sidebar just yet.
+
--[[smcv]]
+
+> Feedback desired. Tagging this [[reviewed]] to take it off the patches list --[[smcv]]