allow for additional props on nonIdealState

[bvandercar-vt]

Fixes #0000

Checklist

• Includes tests
• Update documentation

Changes proposed in this pull request:

Allow additional html props to be passed to nonIdealState. Allows user to set role, or other props.

[adidahiya]

this doesn't work in TypeScript:

[bvandercar-vt]

this doesn't work in TypeScript:

update the props interface, maybe that will fix that

[adidahiya]

maybe that will fix that

I think it will, but just to be sure, it's good practice to add code to this repo which demonstrates usage of your new feature, and checks that it compiles correctly at build time. Please add an representative example of this new feature to one of the NonIdealState examples, perhaps role if you think that's useful in your application.

[bvandercar-vt]

maybe that will fix that

I think it will, but just to be sure, it's good practice to add code to this repo which demonstrates usage of your new feature, and checks that it compiles correctly at build time. Please add an representative example of this new feature to one of the NonIdealState examples, perhaps role if you think that's useful in your application.

great point. added to one of the examples.

[adidahiya]

this example does not show an urgent, time-sensitive message, so this role is inappropriate.

to be honest I don't see any role in the list here which would be appropriate for a NonIdealState. so that begs the question, why are we adding this feature or demonstrating a <NonIdealState> example using the role attribute?

[bvandercar-vt]

we can just cancel this PR