ariaHidden(false, ..) should (probably) use modalNode insted of mountNode

[ruslansavenok]

ariaHidden is added to modal div, not it's container.

But this line tries to remove it from container.

react-overlays/src/ModalManager.js

Line 141 in 4498bb1

ariaHidden(false, data.modals[data.modals.length - 1].mountNode);

[omniverse]

Yes! This is messing up screen readers.

[omniverse]

Actually, the code above is obsolete. But there are still issues with aria-hidden="true" getting set on the dialog itself.

The hideSiblings param declaration hideSiblings(container, { root, backdrop }) doesn't match what is being passed in.
On https://github.com/react-bootstrap/react-overlays/blob/master/src/ModalManager.js#L93 should be something like hideSiblings(container, {root: modal.dialog});

[cameron-martin]

Actually, the code above is obsolete.

This is in the version used by the latest stable version of react-bootstrap. Would you consider releasing a patch for this on that basis?