Bug - SimpleList: arialabel should be applied to inner ul element

[thatblindgeye]

Describe the problem
When passing in aria-label to the SimpleList component, the attribute gets applied to the outer div. This causes axe to flag it as aria labels aren't well supported on div elements.

The aria-label should instead be applied to the ul element that gets rendered instead (currently the isGrouped variation applies an aria-labelledby, so this fix would most likely only need to be applied at line ~77 of the SimpleList code:

patternfly-react/packages/react-core/src/components/SimpleList/SimpleList.tsx

Lines 75 to 78 in d68f302

	<div className={css(styles.simpleList, className)} {...props}>
	{isGrouped && children}
	{!isGrouped && <ul>{children}</ul>}
	</div>

[erkanercan]

Hi @thatblindgeye, can I get assigned for this issue?

[thatblindgeye]

@erkanercan you got it!

[thatblindgeye]

@erkanercan We're looking to get this in with our current milestone so just wanted to check in to see if you'd still be able to work on this issue?

[erkanercan]

Hi @thatblindgeye,
I'll provide a PR for this issue shortly, sorry for the latency.