-
Notifications
You must be signed in to change notification settings - Fork 9
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
Fixes round 3 #1063
Fixes round 3 #1063
Conversation
@@ -68,6 +68,9 @@ extension AssetsView { | |||
} | |||
} | |||
} | |||
#if !DEBUG |
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.
In DEBUG the default value will be used, which is false
. So Fiat value will always be shown in DEBUG mode.
@@ -465,7 +468,7 @@ extension ResourceBalanceView { | |||
} | |||
|
|||
private var useSpacer: Bool { | |||
amount != nil || fallback != nil || caption1 == nil |
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.
Wasn't a good place for the fix, it did push out completely the text field for amount entry.
@@ -15,6 +15,12 @@ extension ResourceAddress { | |||
static let mainnetXRDAddress: ResourceAddress = .xrd(on: .mainnet) | |||
} | |||
|
|||
extension SpecificAddress { | |||
var isOnMainnet: Bool { | |||
networkID == NetworkID.mainnet |
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.
Please out put this in another file. Please :D RadixWallet/EngineKit/
is verboten! :D
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.
ohh yes, forgoten it is verboten :D
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.
LGTM, just move the EngineKit
computed property
Final-final-v6 round of bug fixing...
Fix for AdHoc reported bugs:
Demo
RPReplay_Final1711463312.MP4