Aria selected and row attributes

[louiebert]

Added further capabilities for rendering aria attributes

Description

Added a prop onto the Table for if it is multi-selectable. Additionally, added a callback function for adding DOM element attributes to the row div.

Motivation and Context

Creates more support for screenreaders and user customization.
Fixes #490

How Has This Been Tested?

Given that there aren't any existing tests on DOM attributes rendered, or an installed library that would do so, I did not test these changes.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.

[pradeepnschrodinger]

@wcjordan , should we also have rowAttributesGetter work for the headers (including group header) ? If so, how should it work? Maybe it can specify the first argument with -1 and -2 for the header and group header respectively. Or maybe something like "header", "group-header".

And how should it work with multi level column headers (which will come in v2) ?

[wcjordan]

@wcjordan , should we also have rowAttributesGetter work for the headers (including group header) ? If so, how should it work? Maybe it can specify the first argument with -1 and -2 for the header and group header respectively. Or maybe something like "header", "group-header".

And how should it work with multi level column headers (which will come in v2) ?

I'm not too sure. For now I recommend we leave it off, and wait to see if someone requests it down the road.

[pradeepnschrodinger]

thanks, looking good from my side

[wcjordan]

This looks good to me. @pradeepnschrodinger I think we can go ahead and merge unless there's something you'd like us to look further at.

[louiebert]

@pradeepnschrodinger this hasn't gotten any attention in a little while. Is everything good to merge?

[pradeepnschrodinger]

Sorry for the delays :(

Released with v1.0.1