helpscout / hsds-react / 1536

100%

master: 94%

Pull #454

travis-ci

PropProvider Context + Portal Fixes

This update fixes the transfer for `PropProvider`'s context to content rendered
with `Portal`.

During a React 16 update, we discovered that `PropProvider` values defined
outside of the portal'ed content were not being transferred. This is
due to how `ReactDOM.unstable_renderSubtreeIntoContainer` renders the content
vs. React 16's new context system.

The solution was to create a new HOC (called `contextConnect`), which provides
a component directly with the values from the closest `PropProvider`. Enhancing
`Portal` with this new HOC, I'm able to pass the values through the children
elements. Things work as expected!

3228 of 3228 branches covered (100.0%)

Branch coverage included in aggregate %.

86 of 86 new or added lines in 6 files covered. (100.0%)

6044 of 6044 relevant lines covered (100.0%)

260.02 hits per line