Add <LoadingIndicator onClick> prop, allowing to trigger actions (like a refresh) on click

[david-bezero]

As requested in #9418

This makes it possible to invoke a callback when a user clicks the LoadingIndicator to trigger a refresh, primarily to address this use-case: #3617 (comment)

I didn't find any existing unit tests for this part of the code, but I'm happy to add a unit test if somebody can point me to where it should go.

[fzaninotto]

That's indeed a very minor change that we can merge. Yet it's a new feature, so you should PR against next instead of master.

Also, TypeScript complains.

[david-bezero]

I have fixed the types and updated to point to next.

To avoid going out-of-sync in the future, I'm just forwarding the type from RefreshIconButtonProps (technically the type it expects is ((e: MouseEvent) => void) & MouseEventHandler<HTMLButtonElement>, which looks like a mistake — it should probably be MouseEventHandler<HTMLButtonElement> — but that's a separate issue in RefreshIconButton)

I followed the existing pattern for merging props from RefreshIconButton.

[slax57]

LGTM, thanks!

[fzaninotto]

Thanks!