[Feature Request]: ariaLabel removal or deprecation log changes

[ev-codes]

The problem

getSelectionProps() (part of @carbon/react) returns ariaLabel as one of the properties.

However, <TableSelectAll> logs a deprecation warning to console.warn when ariaLabel is passed into it. Unnecessary logs make it more difficult to find more important issues and slow down CI workflows.

The solution

Either:

• getSelectionProps() no longer returns ariaLabel
• <TableSelectAll> should not log a deprecation warning when receiving ariaLabel

Examples

Code:

<TableSelectAll {...getSelectionProps()} aria-label="entity table select all" />

Logs from test:

console.warn
    Warning: This prop syntax has been deprecated. Please use the new `aria-label`.

Application/PAL

No response

Business priority

None

Available extra resources

No response

Code of Conduct

• I agree to follow this project's Code of Conduct

[github-actions]

Thank you for submitting a feature request. Your proposal is open and will soon be triaged by the Carbon team.

If your proposal is accepted and the Carbon team has bandwidth they will take on the issue, or else request you or other volunteers from the community to work on this issue.

[sstrubberg]

Hey @ev-codes, the warning is intentional and it meant to let you know that you need to change aria-label from ariaLabel. Stop by office hours if you need any more clarity!

[AlanGreene]

@sstrubberg As far as I understand it, the issue reported here is that the instance of ariaLabel triggering the warning is originating from Carbon's own implementation of the getSelectionProps function.

See for example

carbon/packages/react/src/components/DataTable/DataTable.tsx

Lines 165 to 166 in 647881b

	ariaLabel: string;
	'aria-label': string;

The values returned include both ariaLabel and the expected aria-label.

A similar issue exists for a number of the other functions, e.g. getRowsProps, getExpandHeaderProps, etc.

Please reopen this issue.

[tay1orjones]

Yeah, sorry for the confusion, this is something we need to fix in all the applicable DataTable prop getters.

<TableSelectAll> and the other DataTable components are correct in logging a deprecation warning for ariaLabel. aria-label should be used instead.

I'm not sure why the initial deprecation work didn't modify getSelectionProps() to no longer return ariaLabel and use aria-label instead.