-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix multiple issues #6075
Fix multiple issues #6075
Changes from 6 commits
8b40c59
82de44d
c73908f
7051180
321eb41
f0d294e
19d6a15
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -118,6 +118,7 @@ | |
"manageUTXO": "manage UTXO", | ||
"manageKeys": "Manage Keys", | ||
"manageKeysDesc": "Securely manage your keys with ease", | ||
"moreOption": "More Option", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please delete |
||
|
||
"forThisVault": "for this vault", | ||
"saveChanges": "Save Changes", | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -57,17 +57,17 @@ const useWalletAsset = () => { | |
case VaultType.COLLABORATIVE: | ||
return [ | ||
{ tag: 'Collaborative', color: Colors.mintGreen }, | ||
{ tag: getSchemeTag(wallet as Vault), color: Colors.LightMossGreen }, | ||
{ tag: getSchemeTag(wallet as Vault), color: Colors.DesertSand }, | ||
]; | ||
case VaultType.ASSISTED: | ||
return [ | ||
{ tag: 'Assisted', color: Colors.LightMossGreen }, | ||
{ tag: getSchemeTag(wallet as Vault), color: Colors.mintGreen }, | ||
{ tag: getSchemeTag(wallet as Vault), color: Colors.DesertSand }, | ||
]; | ||
case VaultType.TIMELOCKED: | ||
return [ | ||
{ tag: 'Timelocked', color: Colors.mintGreen }, | ||
{ tag: getSchemeTag(wallet as Vault), color: Colors.LightMossGreen }, | ||
{ tag: getSchemeTag(wallet as Vault), color: Colors.DesertSand }, | ||
]; | ||
case VaultType.CANARY: | ||
return [{ tag: 'Canary', color: Colors.LightMossGreen }]; | ||
|
@@ -76,12 +76,12 @@ const useWalletAsset = () => { | |
wallet.scheme.miniscriptScheme?.usedMiniscriptTypes.includes( | ||
MiniscriptTypes.INHERITANCE | ||
) && { tag: 'Inheritance Key', color: Colors.Aquamarine }, | ||
{ tag: getSchemeTag(wallet as Vault), color: Colors.LightMossGreen }, | ||
{ tag: getSchemeTag(wallet as Vault), color: Colors.DesertSand }, | ||
].filter(Boolean); | ||
default: | ||
return [ | ||
{ tag: 'Multi-key', color: Colors.mintGreen }, | ||
{ tag: getSchemeTag(wallet as Vault), color: Colors.LightMossGreen }, | ||
{ tag: 'Multi-key', color: Colors.LabelLight3 }, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It's not enough to assign the colors like this, the colors need to be consistent, ie first pill color is the same for all then second is also same for all etc, |
||
{ tag: getSchemeTag(wallet as Vault), color: Colors.DesertSand }, | ||
]; | ||
} | ||
} else { | ||
|
@@ -94,8 +94,11 @@ const useWalletAsset = () => { | |
derivationPath && WalletUtilities.getPurpose(derivationPath) === DerivationPurpose.BIP86; | ||
|
||
return [ | ||
{ tag: walletKind, color: Colors.LightMossGreen }, | ||
{ tag: isTaprootWallet ? 'Taproot' : 'Single-Key', color: Colors.PaleCyan }, | ||
{ tag: walletKind, color: Colors.DesertSand }, | ||
{ | ||
tag: isTaprootWallet ? 'Taproot' : 'Single-Key', | ||
color: isTaprootWallet ? Colors.PaleCyan : Colors.LabelLight3, | ||
}, | ||
]; | ||
} | ||
}; | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -9,7 +9,7 @@ import ToastErrorIcon from 'src/assets/images/toast_error.svg'; | |
|
||
import KeysIcon from 'src/assets/images/homeGreenKeyIcon.svg'; | ||
import ConciergeIcon from 'src/assets/images/faq-green.svg'; | ||
import SettingIcon from 'src/assets/images/settingsGreenIcon.svg'; | ||
import SettingIcon from 'src/assets/images/more-green.svg'; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Return to the gear icon |
||
import { resetRealyWalletState } from 'src/store/reducers/bhr'; | ||
import InititalAppController from './InititalAppController'; | ||
import { LocalizationContext } from 'src/context/Localization/LocContext'; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This does not look good, please leave the header with the previous title and gear icon