[jira] Updated: (TRINIDAD-1696) acc (screen reader mode) layout tables should include role="presentation"

["Dave Robinson (JIRA)" <de...@myfaces.apache.org>]

     [ https://issues.apache.org/jira/browse/TRINIDAD-1696?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dave Robinson updated TRINIDAD-1696:
------------------------------------

    Status: Patch Available  (was: Open)

> 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
>            Priority: Minor
>   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.