ul role=listbox should define an aria-label or aria-labelledby

[fcabreraupgrade]

Short description

I couldn't define aria-label / aria-labelledby on the item with role=listbox.

For an example of listbox behavior I'm following this example.

As an extra I found defining an accessible name is required for role=listbox aria-docs.

Expected results

We should be able to define props for the element with role=listbox.

Impacted code

https://github.com/ubilabs/react-geosuggest/blob/master/src/suggest-list.tsx#L70-L74

[Borrisson]

Also to add to this issue.

if an initialValue is passed, the <ul> has no <li role="option"> child. If possible either make a conditional where if there are no <li> to list then do not render the ul element.

<div class="geosuggest__suggests-wrapper"><ul class="geosuggest__suggests geosuggest__suggests--hidden" role="listbox" id="geosuggest__list"></ul></div>

[joaogarin]

Opened a PR for this issue, appreciate any feedback 👍