Issue 1:
Pop over on ios safari doesn’t disallow scrolling and this change the intended behavior if someone scrolls down while pop up is on

Expected Behavior

popover to be removed before scrolling is allowed

Behavior

If you scroll on ios safari browser while popover is on the popover doesn’t disappear

Steps to Reproduce (for bugs)

use any demo app with popover you can check it on the demo page

Issue 2:
I have searched the issues of this repository and believe that this is not a duplicate. (keywords searched: hot load muiFormControl Input)

Expected Behavior

When using react-hot-loader in an application with MUI form controls, such as TextField, I expect that making a change to my source code will trigger a successful hot reload.

Current Behavior

Making a change to my source code works fine until i add a TextField to my component.

Once there's a TextField visible, any changes that trigger a hot reload will cause the page to fail to hot-reload, showing errors in the console:

Uncaught TypeError: Cannot read property 'muiFormControl' of undefined
at formControlState (Input.js:264)
at ProxyComponent.componentWillUpdate (Input.js:372)
at next (hotReplacementRender.js:204)
at hotReplacementRender.js:229
at Array.forEach (<anonymous>)
at hotReplacementRender (hotReplacementRender.js:191)
at next (hotReplacementRender.js:207)
at hotReplacementRender.js:229
at Array.forEach (<anonymous>)
at hotReplacementRender (hotReplacementRender.js:191)
at next (hotReplacementRender.js:207)
at hotReplacementRender.js:216
at Array.forEach (<anonymous>)
at hotReplacementRender (hotReplacementRender.js:191)
at next (hotReplacementRender.js:207)
at hotReplacementRender.js:229
at Array.forEach (<anonymous>)
at hotReplacementRender (hotReplacementRender.js:191)
at next (hotReplacementRender.js:207)
at hotReplacementRender.js:229
at Array.forEach (<anonymous>)
at hotReplacementRender (hotReplacementRender.js:191)
at next (hotReplacementRender.js:207)
at hotReplacementRender.js:229
at Array.forEach (<anonymous>)
at hotReplacementRender (hotReplacementRender.js:191)
at next (hotReplacementRender.js:207)
at hotReplacementRender.js:216
at Array.forEach (<anonymous>)
at hotReplacementRender (hotReplacementRender.js:191)
at next (hotReplacementRender.js:207)
at hotReplacementRender.js:229
at Array.forEach (<anonymous>)
at hotReplacementRender (hotReplacementRender.js:191)
at next (hotReplacementRender.js:207)
at hotReplacementRender.js:229
at Array.forEach (<anonymous>)
at hotReplacementRender (hotReplacementRender.js:191)
at next (hotReplacementRender.js:207)
at hotReplacementRender.js:229
at Array.forEach (<anonymous>)
at hotReplacementRender (hotReplacementRender.js:191)
at next (hotReplacementRender.js:207)
at hotReplacementRender.js:216
at Array.forEach (<anonymous>)
at hotReplacementRender (hotReplacementRender.js:191)
at next (hotReplacementRender.js:207)
at hotReplacementRender.js:229
at Array.forEach (<anonymous>)
at hotReplacementRender (hotReplacementRender.js:191)
index.js:2177 The above error occurred in the <AppContainer> component:
in AppContainer (created by HotExportedWithStyles(Component))
in HotExportedWithStyles(Component)
in Router (created by ConnectedRouter)
in ConnectedRouter
in Provider (at index.js:20)
in MuiThemeProvider (at index.js:19)

Steps to Reproduce (for bugs)

could not get codesandbox to work with hot loader so including zipped client app.

ClientApp.zip

yarn install

yarn start

click on New List at top right (drawer should open)

make a change to JSS somewhere in a component (IssueListEditor.js for example)

watch react try to reload and page go blank, then observe errors in console.

Repeat the whole process after commenting out the <TextField/> on line 109 of IssueListEditor.js.

You should not be able to repro the broken hot-load without it.

Context

react hot loading is a basic part of the dev workflow for me personally and at my company. we also love material-ui's look-and-feel, flexibility, and customizability. But not being able to hot-reload when there are form fields in the app is a major problem for development.

Your Environment

osx

yarn: 1.2.3

node: 8.9.3

npm: 4.6.1

react-scripts: 1.0.17

react-hot-loader: 4.0.0-beta.14

package.json.zip

Tech
Version

Material-UI
1.0.0-beta.30

React
16.2.0

browser
chrome latest