From 99be6e17caee1371eec86bd3232fc0d4600979cf Mon Sep 17 00:00:00 2001 From: sdtsui Date: Tue, 15 Aug 2017 05:46:41 +0200 Subject: [WiP] Add comments for multiple modals --- ui/app/components/modals/buy-modal.js | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) diff --git a/ui/app/components/modals/buy-modal.js b/ui/app/components/modals/buy-modal.js index 9a3e4dff9..c69433b1f 100644 --- a/ui/app/components/modals/buy-modal.js +++ b/ui/app/components/modals/buy-modal.js @@ -21,3 +21,23 @@ BuyModal.prototype.render = function () { } // TODO: specify default props and proptypes + +// Generalize to multiple modals: +// Modal API: +// - props { +// key: ['BUY', 'EDIT_ACCOUNT_NAME', 'ACCOUNT_DETAILS'] +// } +// - These props will be passed as 'active' +// mapStateToProps(state, ownProps) { +// active: state.appState.modal[key] +// } +// - Modal accepts: +// - mobileModalStyles, for mobile viewports +// - laptopModalStyles, for laptop viewports +// - backdropStyles +// - Do not set defaults, they are unneeded here +// +// If multiple-step modals are needed: +// - pass a component with internal state that tracks buy steps +// - steps could technically be in redux +// - it renders and does not trigger open/close actions until done \ No newline at end of file -- cgit