From c0d6dcff00b36e89242695a4a29c5f52f4d3a5be Mon Sep 17 00:00:00 2001 From: Kevin Serrano Date: Wed, 5 Oct 2016 02:42:19 -0700 Subject: [PATCH] Rewritten to react standards. Way easier. --- test/integration/tests.js | 2 +- ui/app/first-time/disclaimer.js | 17 ++++++++--------- 2 files changed, 9 insertions(+), 10 deletions(-) diff --git a/test/integration/tests.js b/test/integration/tests.js index a3f3cd294..f8c1a6fd9 100644 --- a/test/integration/tests.js +++ b/test/integration/tests.js @@ -11,7 +11,7 @@ QUnit.test('agree to terms', function (assert) { wait().then(function() { var title = app.find('h1').text() - assert.equal(title, 'MetaMask', 'title screen') + assert.equal(title, 'Terms of Use Version 1', 'title screen') var buttons = app.find('button') assert.equal(buttons.length, 2, 'two buttons: create and restore') diff --git a/ui/app/first-time/disclaimer.js b/ui/app/first-time/disclaimer.js index 312c864d7..3ec0b37c3 100644 --- a/ui/app/first-time/disclaimer.js +++ b/ui/app/first-time/disclaimer.js @@ -19,6 +19,9 @@ function DisclaimerScreen () { } DisclaimerScreen.prototype.render = function () { + const state = this.state || {disclaimerDisabled: true} + const disabled = state.disclaimerDisabled + return ( h('.flex-column.flex-center.flex-grow', [ @@ -67,15 +70,11 @@ DisclaimerScreen.prototype.render = function () { h('div.markdown', { onScroll: (e) => { var object = e.currentTarget - var button = document.getElementById('agree') - if ((object.offsetHeight + object.scrollTop + 100 >= object.scrollHeight) && button.disabled) { - button.disabled = false - button.innerHTML = 'I Agree' - button.addEventListener('click', () => this.props.dispatch(actions.agreeToDisclaimer())) + if (object.offsetHeight + object.scrollTop + 100 >= object.scrollHeight) { + this.setState({disclaimerDisabled: false}) } }, style: { - // whiteSpace: 'pre-line', background: 'rgb(235, 235, 235)', height: '310px', padding: '6px', @@ -91,11 +90,11 @@ DisclaimerScreen.prototype.render = function () { ]), - h('button#agree', { + h('button', { style: { marginTop: '18px' }, - disabled: true, + disabled: disabled, onClick: () => this.props.dispatch(actions.agreeToDisclaimer()), - }, 'Scroll Down to Enable'), + }, disabled ? 'Scroll Down to Enable' : 'I Agree'), ]) ) }