diff --git a/ui/app/app.js b/ui/app/app.js index 5c594c8cf..3ffa9f0d5 100644 --- a/ui/app/app.js +++ b/ui/app/app.js @@ -98,10 +98,7 @@ App.prototype.render = function () { h('.flex-column.full-height', { style: { - // Windows was showing a vertical scroll bar: overflowX: 'hidden', - // TODO: check with dev who committed L75, see if this still happens, and whether auto is enough - // overflowY: 'auto', position: 'relative', alignItems: 'center', }, @@ -140,9 +137,6 @@ App.prototype.renderGlobalModal = function() { } App.prototype.renderSidebar = function() { - // if (!this.props.sidebarOpen) { - // return null; - // } return h('div', { }, [ @@ -170,7 +164,7 @@ App.prototype.renderSidebar = function() { transitionEnterTimeout: 300, transitionLeaveTimeout: 200, }, [ - // content + // A second instance of Walletview is used for non-mobile viewports this.props.sidebarOpen ? h(WalletView, { responsiveDisplayClassname: '.sidebar', style: {}, @@ -502,10 +496,6 @@ App.prototype.renderPrimary = function () { }) } - // Note for @Zanibar - isUnlocked stays true. - // console.log("props.isUnlocked", props.isUnlocked) - // console.log("props.isUnlocked", props.currentView.name) - // show current view switch (props.currentView.name) { diff --git a/ui/app/components/buy-options.js b/ui/app/components/buy-options.js index f0864d087..d36328efd 100644 --- a/ui/app/components/buy-options.js +++ b/ui/app/components/buy-options.js @@ -26,10 +26,12 @@ function BuyOptions () { module.exports = connect(mapStateToProps, mapDispatchToProps)(BuyOptions) +// BuyOptions is currently meant to be rendered inside +// It is the only component in this codebase that does so +// It utilizes modal styles BuyOptions.prototype.render = function () { return h('div', {}, [ h('div.modal-content.transfers-subview', { - }, [ h('div.modal-content-title-wrapper.flex-column.flex-center', { style: {}, @@ -44,7 +46,6 @@ BuyOptions.prototype.render = function () { h('div.modal-content-option', { onClick: () => { - console.log("buy clicked") const { toCoinbase, address } = this.props toCoinbase(address) }, diff --git a/ui/app/components/modal.js b/ui/app/components/modal.js index 2339ee497..8bac866a8 100644 --- a/ui/app/components/modal.js +++ b/ui/app/components/modal.js @@ -1,10 +1,7 @@ const Component = require('react').Component - const h = require('react-hyperscript') const inherits = require('util').inherits const connect = require('react-redux').connect -// const elementType = require('react-prop-types/lib/elementType') -// const PropTypes from 'prop-types' const FadeModal = require('boron').FadeModal const actions = require('../actions') const isMobileView = require('../../lib/is-mobile-view') @@ -30,16 +27,16 @@ function Modal () { module.exports = connect(mapStateToProps, mapDispatchToProps)(Modal) -Modal.prototype.render = function () { +const mobileModalStyles = { + width: '95%', +} - const mobileModalStyles = { - width: '95%', - } +const laptopModalStyles = { + width: '66%', + top: '30%', +} - const laptopModalStyles = { - width: '66%', - top: '30%', - } +Modal.prototype.render = function () { return h(FadeModal, { @@ -78,8 +75,12 @@ Modal.prototype.show = function() { this.modalRef.show() } +// TODO: specify default props and proptypes // Modal.defaultProps = {} +// const elementType = require('react-prop-types/lib/elementType') +// const PropTypes from 'prop-types' + // Modal.propTypes = { // active: PropTypes.bool, // hideModal: PropTypes.func.isRequired,