fix: 🐛 [JIRA: HCPSDKFIORIUIKIT-1940] profile header style (#744)

Co-authored-by: Bill Zhou <bill.zhou01@sap.com> Co-authored-by: dyongxu <61523257+dyongxu@users.noreply.github.com>
SAP · Jul 22, 2024 · 585f412 · 585f412
1 parent cb4a0b9
commit 585f412
Show file tree

Hide file tree

Showing 2 changed files with 38 additions and 11 deletions.
diff --git a/Apps/Examples/Examples/FioriSwiftUICore/ProfileHeader/ProfileHeaderStaticExample.swift b/Apps/Examples/Examples/FioriSwiftUICore/ProfileHeader/ProfileHeaderStaticExample.swift
@@ -2,16 +2,14 @@ import FioriSwiftUICore
 import SwiftUI
 
 struct ProfileHeaderStaticExample: View {
-    var profileHeader: some View {
-        ProfileHeader(detailImage: {
-            Image("ProfilePic").resizable()
-        }, title: {
-            Text("Title")
-        }, subtitle: {
-            Text("This is a subtitle.")
-        }, description: {
-            Text("This is a description.")
-        }) {
+    @State var useButtonForDetails = false
+
+    @ViewBuilder var detailContent: some View {
+        if self.useButtonForDetails {
+            FioriButton { _ in
+                Text("Label")
+            }
+        } else {
             HStack(spacing: 30) {
                 Button(action: {}, label: {
                     Image(systemName: "mail")
@@ -27,6 +25,33 @@ struct ProfileHeaderStaticExample: View {
         }
     }
 
+    var profileHeader: some View {
+        ProfileHeader(detailImage: {
+            Image("ProfilePic").resizable()
+        }, title: {
+            Text("Title")
+        }, subtitle: {
+            Text("This is a subtitle.")
+        }, description: {
+            Text("This is a description.")
+        }) {
+            self.detailContent
+        }
+        .toolbar {
+            ToolbarItem(placement: .topBarTrailing) {
+                Menu {
+                    Button(action: {
+                        self.useButtonForDetails.toggle()
+                    }, label: {
+                        Text("Icons / Button")
+                    })
+                } label: {
+                    Text("Styles")
+                }
+            }
+        }
+    }
+
     var body: some View {
         ScrollView {
             LazyVStack(alignment: .center, pinnedViews: [.sectionHeaders], content: {

diff --git a/Sources/FioriSwiftUICore/_FioriStyles/ProfileHeaderStyle.fiori.swift b/Sources/FioriSwiftUICore/_FioriStyles/ProfileHeaderStyle.fiori.swift
@@ -195,6 +195,7 @@ extension ProfileHeaderFioriStyle {
 
         func makeBody(_ configuration: TitleConfiguration) -> some View {
             Title(configuration)
+                .foregroundStyle(Color.preferredColor(.primaryLabel))
                 .font(.fiori(forTextStyle: .title3, weight: .bold))
         }
     }
@@ -204,7 +205,8 @@ extension ProfileHeaderFioriStyle {
 
         func makeBody(_ configuration: SubtitleConfiguration) -> some View {
             Subtitle(configuration)
-                .font(.fiori(forTextStyle: .body))
+                .foregroundStyle(Color.preferredColor(.secondaryLabel))
+                .font(.fiori(forTextStyle: .headline))
         }
     }