search

modxcms / a11y

MODX Accessibility for the Manager
14 stars 10 forks source link

Main role element #60

Closed dubrod closed 8 years ago

dubrod commented 8 years ago

need <main role="main"> around ext-gen17 after #modx-panel-holder

dubrod commented 8 years ago

Hey @theboxer @kensgists @paulbohman
i wrapped the main element around whatever is created between header.tpl and footer.tpl

The resulting output is 

<main role="main" id="ext-gen45">
<div id="modx-panel-holder"></div><div id="modx-panel-resource-div"></div>
</main>

BUT

highlighting the element in the inspector shows that the MODX Ext JS output is mainly in element <div class="x-panel-bwrap" id="ext-gen17"> which is just below it.

So I'm not sure this change helps the cause. Needs testing