mirror of
https://github.com/kremalicious/metamask-extension.git
synced 2024-12-23 01:39:44 +01:00
[NewUI] Hide UI toggle in mascara (#2772)
* Hides old-UI on mascara. * Improve code clarity in select-app.js
This commit is contained in:
parent
109e4e5d96
commit
bccbf14b39
@ -405,7 +405,7 @@ App.prototype.renderDropdown = function () {
|
||||
|
||||
h(DropdownMenuItem, {
|
||||
closeMenu: () => this.setState({ isMainMenuOpen: !isOpen }),
|
||||
onClick: () => { this.props.dispatch(actions.setFeatureFlag('betaUI', true)) },
|
||||
onClick: () => { this.props.dispatch(actions.setFeatureFlag('betaUI', true, 'BETA_UI_NOTIFICATION_MODAL')) },
|
||||
}, 'Try Beta!'),
|
||||
])
|
||||
}
|
||||
|
@ -1523,10 +1523,7 @@ function updateTokenExchangeRate (token = '') {
|
||||
}
|
||||
}
|
||||
|
||||
function setFeatureFlag (feature, activated) {
|
||||
const notificationType = activated
|
||||
? 'BETA_UI_NOTIFICATION_MODAL'
|
||||
: 'OLD_UI_NOTIFICATION_MODAL'
|
||||
function setFeatureFlag (feature, activated, notificationType) {
|
||||
return (dispatch) => {
|
||||
dispatch(actions.showLoadingIndication())
|
||||
return new Promise((resolve, reject) => {
|
||||
@ -1537,7 +1534,7 @@ function setFeatureFlag (feature, activated) {
|
||||
reject(err)
|
||||
}
|
||||
dispatch(actions.updateFeatureFlags(updatedFeatureFlags))
|
||||
dispatch(actions.showModal({ name: notificationType }))
|
||||
notificationType && dispatch(actions.showModal({ name: notificationType }))
|
||||
resolve(updatedFeatureFlags)
|
||||
})
|
||||
})
|
||||
|
@ -12,35 +12,40 @@ function mapStateToProps (state) {
|
||||
betaUI: state.metamask.featureFlags.betaUI,
|
||||
autoAdd: autoAddToBetaUI(state),
|
||||
isUnlocked: state.metamask.isUnlocked,
|
||||
isMascara: state.metamask.isMascara,
|
||||
}
|
||||
}
|
||||
|
||||
function mapDispatchToProps (dispatch) {
|
||||
return {
|
||||
setFeatureFlagToBeta: () => dispatch(setFeatureFlag('betaUI', true)),
|
||||
setFeatureFlagWithModal: () => dispatch(setFeatureFlag('betaUI', true, 'BETA_UI_NOTIFICATION_MODAL')),
|
||||
setFeatureFlagWithoutModal: () => dispatch(setFeatureFlag('betaUI', true)),
|
||||
}
|
||||
}
|
||||
module.exports = connect(mapStateToProps, mapDispatchToProps)(SelectedApp)
|
||||
|
||||
inherits(SelectedApp, Component)
|
||||
function SelectedApp () {
|
||||
this.state = {
|
||||
autoAdd: false,
|
||||
}
|
||||
Component.call(this)
|
||||
}
|
||||
|
||||
SelectedApp.prototype.componentWillReceiveProps = function (nextProps) {
|
||||
const { isUnlocked, setFeatureFlagToBeta } = this.props
|
||||
const {
|
||||
isUnlocked,
|
||||
setFeatureFlagWithModal,
|
||||
setFeatureFlagWithoutModal,
|
||||
isMascara,
|
||||
} = this.props
|
||||
|
||||
if (!isUnlocked && nextProps.isUnlocked && nextProps.autoAdd) {
|
||||
this.setState({ autoAdd: nextProps.autoAdd })
|
||||
setFeatureFlagToBeta()
|
||||
if (isMascara) {
|
||||
setFeatureFlagWithoutModal()
|
||||
} else if (!isUnlocked && nextProps.isUnlocked && (nextProps.autoAdd)) {
|
||||
setFeatureFlagWithModal()
|
||||
}
|
||||
}
|
||||
|
||||
SelectedApp.prototype.render = function () {
|
||||
const { betaUI } = this.props
|
||||
const Selected = betaUI ? App : OldApp
|
||||
const { betaUI, isMascara } = this.props
|
||||
const Selected = betaUI || isMascara ? App : OldApp
|
||||
return h(Selected)
|
||||
}
|
||||
|
@ -250,7 +250,7 @@ class Settings extends Component {
|
||||
}
|
||||
|
||||
renderSettingsContent () {
|
||||
const { warning } = this.props
|
||||
const { warning, isMascara } = this.props
|
||||
|
||||
return (
|
||||
h('div.settings__content', [
|
||||
@ -261,7 +261,7 @@ class Settings extends Component {
|
||||
this.renderNewRpcUrl(),
|
||||
this.renderStateLogs(),
|
||||
this.renderSeedWords(),
|
||||
this.renderOldUI(),
|
||||
!isMascara && this.renderOldUI(),
|
||||
])
|
||||
)
|
||||
}
|
||||
@ -386,12 +386,14 @@ Settings.propTypes = {
|
||||
setFeatureFlagToBeta: PropTypes.func,
|
||||
warning: PropTypes.string,
|
||||
goHome: PropTypes.func,
|
||||
isMascara: PropTypes.bool,
|
||||
}
|
||||
|
||||
const mapStateToProps = state => {
|
||||
return {
|
||||
metamask: state.metamask,
|
||||
warning: state.appState.warning,
|
||||
isMascara: state.metamask.isMascara,
|
||||
}
|
||||
}
|
||||
|
||||
@ -403,7 +405,7 @@ const mapDispatchToProps = dispatch => {
|
||||
displayWarning: warning => dispatch(actions.displayWarning(warning)),
|
||||
revealSeedConfirmation: () => dispatch(actions.revealSeedConfirmation()),
|
||||
setUseBlockie: value => dispatch(actions.setUseBlockie(value)),
|
||||
setFeatureFlagToBeta: () => dispatch(actions.setFeatureFlag('betaUI', false)),
|
||||
setFeatureFlagToBeta: () => dispatch(actions.setFeatureFlag('betaUI', false, 'OLD_UI_NOTIFICATION_MODAL')),
|
||||
}
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user