chore(wallet-mobile): drop disable status for action buttons #3846
Conversation
| @@ -22,7 +22,7 @@ import {useSelectedWallet} from '../../../WalletManager/common/hooks/useSelected | |||
| import {useWalletManager} from '../../../WalletManager/context/WalletManagerProvider' | |||
| import {useStrings} from '../../common/strings' | |||
|
|
|||
| export const ActionsBanner = ({disabled = false}: {disabled: boolean}) => { | |||
| export const ActionsBanner = () => { | |||
There was a problem hiding this comment.
disabled was added for the first load, since now it doesnt open the wallet before a proper sync (exception for dev build), it is no longer needed.
There was a problem hiding this comment.
What happened with the check for pool transition?
There was a problem hiding this comment.
is it part of tx loading state?
There was a problem hiding this comment.
I see two things here, 1st possible a clash with another modal if the user open whatever right after loading the UI, but this would happen anyway if any other action has a modal, 2nd all pools were already migrated, so no worries (for now), the 1st issue will be solved only after building a queue for the modal.
stackchain
force-pushed
the
chore/drop-action-bar-disabled
branch
from
0b8da3c to
b5e7188
Compare
stackchain
force-pushed
the
chore/drop-action-bar-disabled
branch
from
1ab7b26 to
3496ff2
Compare
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
Description / Change(s) / Related issue(s)
Ticket
YV-260