git.iko.soy/typeable/elm-ui
on master help sign in
remove role paragraph
742f67bMatthew Griffith
src/Internal/Model.elm+3−2
@@ -1144,15 +1144,16 @@ gatherAttrRecursive classes node has transform styles attrs children elementAttr
1144 -- previously we rendered a <p> tag, though apparently this invalidates the html if it has <div>s inside.
1144 -- previously we rendered a <p> tag, though apparently this invalidates the html if it has <div>s inside.
1145 -- Since we can't guaranteee that there are no divs, we need another strategy.
1145 -- Since we can't guaranteee that there are no divs, we need another strategy.
1146 -- While it's not documented in many places, there apparently is a paragraph aria role
1146 -- While it's not documented in many places, there apparently is a paragraph aria role
1147− -- maybe it doesn't do anything, but it might help.
1148 -- https://github.com/w3c/aria/blob/11f85f41a5b621fdbe85fc9bcdcd270e653a48ba/common/script/roleInfo.js
1147 -- https://github.com/w3c/aria/blob/11f85f41a5b621fdbe85fc9bcdcd270e653a48ba/common/script/roleInfo.js
1148+ -- Though we'll need to wait till it gets released in an official wai-aria spec to use it.
1149+ -- If it's used at the moment, then Lighthouse complains (likely rightfully) that role paragraph is not recognized.
1149 gatherAttrRecursive
1150 gatherAttrRecursive
1150 classes
1151 classes
1151 node
1152 node
1152 has
1153 has
1153 transform
1154 transform
1154 styles
1155 styles
1155− (VirtualDom.attribute "role" "paragraph" :: attrs)
1156+ attrs
1156 children
1157 children
1157 remaining
1158 remaining
1158
1159