Address PR comments

brave · Oct 10, 2023 · 8ff2feb · 8ff2feb
1 parent 6b058b8
commit 8ff2feb
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/Sources/BraveWallet/Crypto/Portfolio/PortfolioHeaderView.swift b/Sources/BraveWallet/Crypto/Portfolio/PortfolioHeaderView.swift
@@ -208,7 +208,8 @@ private struct PortfolioHeaderButton: View {
           .overlay(
             Image(braveSystemName: style.iconName)
               .foregroundColor(.white)
-              .dynamicTypeSize(.large)
+              .font(.title3.weight(.semibold))
+              .dynamicTypeSize(...DynamicTypeSize.xLarge)
           )
         Text(style.label)
           .font(.footnote.weight(.semibold))

diff --git a/Sources/BraveWallet/Crypto/Stores/PortfolioStore.swift b/Sources/BraveWallet/Crypto/Stores/PortfolioStore.swift
@@ -196,7 +196,7 @@ struct BalanceDifference: Equatable {
 public class PortfolioStore: ObservableObject, WalletObserverStore {
   /// The dollar amount of your portfolio
   @Published private(set) var balance: String = "$0.00"
-  ///
+  /// Balance difference used to display difference between first point on the graph and current balance.
   @Published private(set) var balanceDifference: BalanceDifference?
   /// The users visible fungible token groups.
   @Published private(set) var assetGroups: [AssetGroupViewModel] = []