mdehoog / Semantic-UI-Calendar

Calendar module for Semantic UI
MIT License
805 stars 126 forks source link

A11Y issues: data table does not use scope, id and headers to identify cells #90

Open smartm0use opened 7 years ago

smartm0use commented 7 years ago

I'm creating an application that should be compliant with WCAG 2.0 specifications. I have add the calendar module in my Semantic-UI project. If I test the html code with A11Y checker, I get the following issues:

Check 244: Data table with both row and column headers does not use scope to identify cells. Repair: Add scope attributes to header cells so they identify the cells that relate to the header.

Error Line 1530, Column 162:

<table class="ui celled center aligned unstackable table seven column day"><thead scope="col"><tr><t ...

Error Line 1536, Column 162:

<table class="ui celled center aligned unstackable table seven column day"><thead><tr><th colspan="7 ...

Error Line 1548, Column 162:

<table class="ui celled center aligned unstackable table seven column day"><thead><tr><th colspan="7 ...

Error Line 1554, Column 162:

<table class="ui celled center aligned unstackable table seven column day"><thead><tr><th colspan="7 ...

Check 245: Data table with more than one row/column of headers does not use id and headers attributes to identify cells. Repair: Add id and headers attributes to table cells so they identify the cells that relate to the headers.

Error Line 1530, Column 162:

<table class="ui celled center aligned unstackable table seven column day"><thead scope="col"><tr><t ...

Error Line 1536, Column 162:

<table class="ui celled center aligned unstackable table seven column day"><thead><tr><th colspan="7 ...

Error Line 1548, Column 162:

<table class="ui celled center aligned unstackable table seven column day"><thead><tr><th colspan="7 ...

Error Line 1554, Column 162:

<table class="ui celled center aligned unstackable table seven column day"><thead><tr><th colspan="7 ...

Can you upgrade the calendar to make it compliant with A11Y WCAG 2.0 requirements?