Skip to content

Commit

Permalink
Merge pull request #276 from oigewan/aria
Browse files Browse the repository at this point in the history
Fix aria roles on weekday and month elements
  • Loading branch information
gpbl authored Mar 8, 2017
2 parents 9fb17f8 + 36fdfb7 commit f81357b
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions src/Month.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ export default function Month({

const weeks = getWeekArray(month, firstDayOfWeek, fixedWeeks);
return (
<div className={ classNames.month }>
<div className={ classNames.month } role="grid">
{caption}
<Weekdays
classNames={ classNames }
Expand All @@ -47,7 +47,7 @@ export default function Month({
localeUtils={ localeUtils }
weekdayElement={ weekdayElement }
/>
<div className={ classNames.body } role="grid">
<div className={ classNames.body } role="rowgroup">
{
weeks.map((week, j) =>
<div key={ j } className={ classNames.week } role="gridcell">
Expand Down
2 changes: 1 addition & 1 deletion src/Weekday.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ export default function Weekday({
}

return (
<div className={ className }>
<div className={ className } role="columnheader">
<abbr title={ title }>
{content}
</abbr>
Expand Down
2 changes: 1 addition & 1 deletion src/Weekdays.js
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ export default function Weekdays({

return (
<div className={ classNames.weekdays } role="rowgroup">
<div className={ classNames.weekdaysRow } role="columnheader">
<div className={ classNames.weekdaysRow } role="row">
{ days }
</div>
</div>
Expand Down

0 comments on commit f81357b

Please sign in to comment.