[ https://issues.apache.org/jira/browse/TRINIDAD-1696?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Matt Cooper updated TRINIDAD-1696: ---------------------------------- Resolution: Fixed Fix Version/s: 1.2.14-core Status: Resolved (was: Patch Available) > acc (screen reader mode) layout tables should include role="presentation" > ------------------------------------------------------------------------- > > Key: TRINIDAD-1696 > URL: https://issues.apache.org/jira/browse/TRINIDAD-1696 > Project: MyFaces Trinidad > Issue Type: Bug > Affects Versions: 1.2.12-core > Environment: all > Reporter: Dave Robinson > Assignee: Matt Cooper > Priority: Minor > Fix For: 1.2.14-core > > Attachments: TRINIDAD-1696 for trinidad 1.2.12.1.patch, TRINIDAD-1696 > for trinidad 1.2.12.2.patch, TRINIDAD-1696 for trinidad trunk.patch > > Original Estimate: 3h > Remaining Estimate: 3h > > When using trh:tableLayout in our page to layout some UI components, it gives > warning during Accessibility testing: > "WARNING - This layout Table could be confused for a data table by Screen > Readers" > From the html perspective, this warning can be fixed by setting > role="presentation" > on the html table element. > We can add > role="presentation" to layout tables with the following addition to > org.apache.myfaces.trinidadinternal.renderkit.core.xhtml.OutputTextUtils.renderLayoutTableAttributes(): > if (CoreRenderer.isScreenReaderMode(arc)) > { > ResponseWriter writer = context.getResponseWriter(); > writer.writeAttribute("datatable", "0", null); > --> writer.writeAttribute("role", "presentation", null); <-- > } -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.