make Modal always scrollable (#3667)

js/src/modals/DeployContract/deployContract.js

@@ -128,7 +128,7 @@ export default class DeployContract extends Component {
         title={ title }
         waiting={ waiting }
         visible
-        scroll>
+      >
         { this.renderStep() }
       </Modal>
     );

js/src/modals/LoadContract/loadContract.js

@@ -62,7 +62,6 @@ export default class LoadContract extends Component {
         title={ title }
         actions={ this.renderDialogActions() }
         visible
-        scroll
       >
         { this.renderBody() }
       </Modal>

js/src/modals/SMSVerification/SMSVerification.js

@@ -61,7 +61,7 @@ export default class SMSVerification extends Component {
       <Modal
         actions={ this.renderDialogActions(phase, error, isStepValid) }
         title='verify your account via SMS'
-        visible scroll
+        visible
         current={ phase }
         steps={ ['Prepare', 'Enter Data', 'Request', 'Enter Code', 'Confirm', 'Done!'] }
         waiting={ error ? [] : [ 0, 2, 4 ] }

js/src/modals/Transfer/transfer.js

@@ -104,7 +104,6 @@ class Transfer extends Component {
         steps={ steps }
         waiting={ extras ? [2] : [1] }
         visible
-        scroll
       >
         { this.renderWarning() }
         { this.renderPage() }

js/src/ui/Modal/modal.js

@@ -41,7 +41,6 @@ class Modal extends Component {
     compact: PropTypes.bool,
     current: PropTypes.number,
     waiting: PropTypes.array,
-    scroll: PropTypes.bool,
     steps: PropTypes.array,
     title: PropTypes.oneOfType([
       PropTypes.node, PropTypes.string
@@ -52,7 +51,7 @@ class Modal extends Component {
 
   render () {
     const { muiTheme } = this.context;
-    const { actions, busy, className, current, children, compact, scroll, steps, waiting, title, visible, settings } = this.props;
+    const { actions, busy, className, current, children, compact, steps, waiting, title, visible, settings } = this.props;
     const contentStyle = muiTheme.parity.getBackgroundStyle(null, settings.backgroundSeed);
     const header = (
       <Title
@@ -70,7 +69,7 @@ class Modal extends Component {
         actions={ actions }
         actionsContainerStyle={ ACTIONS_STYLE }
         autoDetectWindowHeight={ false }
-        autoScrollBodyContent={ !!scroll }
+        autoScrollBodyContent
         actionsContainerClassName={ styles.actions }
         bodyClassName={ styles.body }
         contentClassName={ styles.content }

js/src/views/Contract/contract.js

@@ -165,7 +165,6 @@ class Contract extends Component {
         actions={ [ cancelBtn ] }
         title={ 'contract details' }
         visible
-        scroll
       >
         <div className={ styles.details }>
           { this.renderSource(contract) }

js/src/views/Dapps/AddDapps/AddDapps.js

@@ -50,7 +50,7 @@ export default class AddDapps extends Component {
           />
         ] }
         visible
-        scroll>
+      >
         <div className={ styles.warning } />
         { this.renderList(store.sortedLocal, 'Applications locally available', 'All applications installed locally on the machine by the user for access by the Parity client.') }
         { this.renderList(store.sortedBuiltin, 'Applications bundled with Parity', 'Experimental applications developed by the Parity team to show off dapp capabilities, integration, experimental features and to control certain network-wide client behaviour.') }