Merge pull request #42 from DatepollSystems/feature/shared-update

TableGroup ProductGroup coloring, shared update
DatepollSystems · Jul 10, 2024 · 06765d6 · 06765d6
2 parents a1e5836 + 4ceaa19
commit 06765d6
Show file tree

Hide file tree

Showing 13 changed files with 247 additions and 44 deletions.
diff --git a/WaiterRobot/Resources/Colors.xcassets/darkRed.colorset/Contents.json b/WaiterRobot/Resources/Colors.xcassets/darkRed.colorset/Contents.json
@@ -0,0 +1,20 @@
+{
+  "colors" : [
+    {
+      "color" : {
+        "color-space" : "display-p3",
+        "components" : {
+          "alpha" : "1.000",
+          "blue" : "0x00",
+          "green" : "0x22",
+          "red" : "0x9B"
+        }
+      },
+      "idiom" : "universal"
+    }
+  ],
+  "info" : {
+    "author" : "xcode",
+    "version" : 1
+  }
+}
diff --git a/WaiterRobot/Resources/Colors.xcassets/lightRed.colorset/Contents.json b/WaiterRobot/Resources/Colors.xcassets/lightRed.colorset/Contents.json
@@ -0,0 +1,20 @@
+{
+  "colors" : [
+    {
+      "color" : {
+        "color-space" : "display-p3",
+        "components" : {
+          "alpha" : "1.000",
+          "blue" : "0x3C",
+          "green" : "0x34",
+          "red" : "0xFF"
+        }
+      },
+      "idiom" : "universal"
+    }
+  ],
+  "info" : {
+    "author" : "xcode",
+    "version" : 1
+  }
+}
diff --git a/WaiterRobot/Ui/Billing/BillingScreen.swift b/WaiterRobot/Ui/Billing/BillingScreen.swift
@@ -21,7 +21,7 @@ struct BillingScreen: View {
         let billItems = Array(viewModel.state.billItemsArray)
 
         content(billItems: billItems)
-            .navigationTitle(localize.billing.title(value0: table.number.description, value1: table.groupName))
+            .navigationTitle(localize.billing.title(value0: table.groupName, value1: table.number.description))
             .navigationBarTitleDisplayMode(.inline)
             .customBackNavigation(
                 title: localize.dialog.cancel(),
@@ -63,7 +63,6 @@ struct BillingScreen: View {
                     }
                 }
             }
-
             // TODO: make only half screen when ios 15 is dropped
             .sheet(isPresented: $showPayDialog) {
                 PayDialog(viewModel: viewModel)
@@ -76,7 +75,7 @@ struct BillingScreen: View {
         VStack {
             List {
                 if billItems.isEmpty {
-                    Text(localize.billing.noOpenBill(value0: table.number.description, value1: table.groupName))
+                    Text(localize.billing.noOpenBill(value0: table.groupName, value1: table.number.description))
                         .multilineTextAlignment(.center)
                         .frame(maxWidth: .infinity)
                         .padding()
@@ -124,6 +123,7 @@ struct BillingScreen: View {
                         .font(.system(.title))
                         .padding()
                         .tint(.white)
+                        .offset(x: -3)
                 }
                 .background(.blue)
                 .mask(Circle())

diff --git a/WaiterRobot/Ui/Order/OrderScreen.swift b/WaiterRobot/Ui/Order/OrderScreen.swift
@@ -38,7 +38,7 @@ struct OrderScreen: View {
                 currentOder(resource.data)
             }
         }
-        .navigationTitle(localize.order.title(value0: table.number.description, value1: table.groupName))
+        .navigationTitle(localize.order.title(value0: table.groupName, value1: table.number.description))
         .navigationBarTitleDisplayMode(.large)
         .navigationBarBackButtonHidden()
         .confirmationDialog(

diff --git a/WaiterRobot/Ui/Order/ProductListItem.swift b/WaiterRobot/Ui/Order/ProductListItem.swift
@@ -3,12 +3,18 @@ import SwiftUI
 
 struct ProductListItem: View {
     let product: Product
+    let backgroundColor: Color?
     let onClick: () -> Void
 
     private let allergens: String
 
-    init(product: Product, onClick: @escaping () -> Void) {
+    init(
+        product: Product,
+        backgroundColor: Color?,
+        onClick: @escaping () -> Void
+    ) {
         self.product = product
+        self.backgroundColor = backgroundColor
         self.onClick = onClick
 
         var allergens = ""
@@ -23,29 +29,42 @@ struct ProductListItem: View {
         }
     }
 
-    var body: some View {
-        ZStack {
-            RoundedRectangle(cornerRadius: 10, style: .continuous)
-                .fill(product.soldOut ? Color.gray.opacity(0.1) : Color(.systemBackground))
-                .shadow(radius: 2)
+    var foregroundColor: Color {
+        if product.soldOut {
+            return .blackWhite
+        }
+
+        if let backgroundColor {
+            return backgroundColor.getContentColor(lightColorScheme: .black, darkColorScheme: .white)
+        } else {
+            return Color.blackWhite
+        }
+    }
 
-            Button {
-                onClick()
-            } label: {
-                VStack {
-                    Text(product.name)
-                        .strikethrough(product.soldOut)
-                    if !product.allergens.isEmpty {
-                        Text(allergens)
-                            .foregroundColor(.gray)
-                    }
-                    Text(product.price.description())
+    var body: some View {
+        Button {
+            onClick()
+        } label: {
+            VStack {
+                Text(product.name)
+                    .strikethrough(product.soldOut)
+                    .foregroundStyle(foregroundColor)
+                if !product.allergens.isEmpty {
+                    Text(allergens)
+                        .foregroundStyle(foregroundColor)
+                        .opacity(0.6)
                 }
-                .foregroundColor(.blackWhite)
-                .frame(maxWidth: .infinity)
-                .padding(5)
+                Text(product.price.description())
             }
-            .disabled(product.soldOut)
+            .foregroundStyle(foregroundColor)
+            .frame(maxWidth: .infinity)
+            .padding(5)
+        }
+        .disabled(product.soldOut)
+        .background {
+            RoundedRectangle(cornerRadius: 10, style: .continuous)
+                .fill(product.soldOut ? Color.gray.opacity(0.1) : backgroundColor ?? Color(.systemBackground))
+                .shadow(radius: 2)
         }
     }
 }
@@ -66,6 +85,30 @@ struct ProductListItem: View {
             ],
             position: 1
         ),
+        backgroundColor: .yellow,
+        onClick: {}
+    )
+    .frame(maxWidth: 100, maxHeight: 100)
+}
+
+#Preview {
+    ProductListItem(
+        product: Product(
+            id: 2,
+            name: "Wine",
+            price: Money(cents: 290),
+            soldOut: false,
+            allergens: [
+                Allergen(id: 1, name: "Egg", shortName: "E"),
+                Allergen(id: 2, name: "Egg2", shortName: "A"),
+                Allergen(id: 3, name: "Egg3", shortName: "B"),
+                Allergen(id: 4, name: "Egg4", shortName: "C"),
+                Allergen(id: 5, name: "Egg5", shortName: "D"),
+            ],
+            position: 1
+        ),
+        backgroundColor: .yellow,
         onClick: {}
     )
+    .frame(maxWidth: 100, maxHeight: 100)
 }
diff --git a/WaiterRobot/Ui/Order/Search/ProductSearch.swift b/WaiterRobot/Ui/Order/Search/ProductSearch.swift
@@ -60,6 +60,7 @@ struct ProductSearch: View {
                             LazyVGrid(columns: layout, spacing: 0) {
                                 ProductSearchGroupList(
                                     products: groupWithProducts.products,
+                                    backgroundColor: Color(hex: groupWithProducts.color),
                                     onProductClick: {
                                         viewModel.actual.addItem(product: $0, amount: 1)
                                         dismiss()

diff --git a/WaiterRobot/Ui/Order/Search/ProductSearchAllTab.swift b/WaiterRobot/Ui/Order/Search/ProductSearchAllTab.swift
@@ -14,6 +14,7 @@ struct ProductSearchAllTab: View {
                         Section {
                             ProductSearchGroupList(
                                 products: productGroup.products,
+                                backgroundColor: Color(hex: productGroup.color),
                                 onProductClick: onProductClick
                             )
                         } header: {
@@ -38,6 +39,7 @@ struct ProductSearchAllTab: View {
                 id: 1,
                 name: "Test Group 1",
                 position: 1,
+                color: "",
                 products: [
                     Product(
                         id: 1,

diff --git a/WaiterRobot/Ui/Order/Search/ProductSearchGroupList.swift b/WaiterRobot/Ui/Order/Search/ProductSearchGroupList.swift
@@ -3,11 +3,12 @@ import SwiftUI
 
 struct ProductSearchGroupList: View {
     let products: [Product]
+    let backgroundColor: Color?
     let onProductClick: (Product) -> Void
 
     var body: some View {
         ForEach(products, id: \.id) { product in
-            ProductListItem(product: product) {
+            ProductListItem(product: product, backgroundColor: backgroundColor) {
                 onProductClick(product)
             }
             .foregroundColor(.blackWhite)
@@ -29,6 +30,7 @@ struct ProductSearchGroupList: View {
                     position: 1
                 ),
             ],
+            backgroundColor: .yellow,
             onProductClick: { _ in }
         )
     }

diff --git a/WaiterRobot/Ui/TableDetail/TableDetailScreen.swift b/WaiterRobot/Ui/TableDetail/TableDetailScreen.swift
@@ -17,7 +17,7 @@ struct TableDetailScreen: View {
 
     var body: some View {
         content()
-            .navigationTitle(localize.tableDetail.title(value0: table.number.description, value1: table.groupName))
+            .navigationTitle(localize.tableDetail.title(value0: table.groupName, value1: table.number.description))
             .withViewModel(viewModel, navigator)
     }
 
@@ -45,7 +45,7 @@ struct TableDetailScreen: View {
             if orderedItems.isEmpty {
                 Spacer()
 
-                Text(localize.tableDetail.noOrder(value0: table.number.description, value1: table.groupName))
+                Text(localize.tableDetail.noOrder(value0: table.groupName, value1: table.number.description))
                     .multilineTextAlignment(.center)
                     .frame(maxWidth: .infinity)
                     .padding()

diff --git a/WaiterRobot/Ui/TableList/TableGroupSection.swift b/WaiterRobot/Ui/TableList/TableGroupSection.swift
@@ -11,6 +11,7 @@ struct TableGroupSection: View {
                 TableView(
                     text: table.number.description,
                     hasOrders: table.hasOrders,
+                    backgroundColor: Color(hex: tableGroup.color),
                     onClick: {
                         onTableClick(table)
                     }
@@ -19,19 +20,27 @@ struct TableGroupSection: View {
             }
         } header: {
             HStack {
-                Text(tableGroup.name)
-                    .font(.title2)
-                    .foregroundStyle(.white)
-                    .padding(6)
-                    .background {
-                        RoundedRectangle(cornerRadius: 8.0)
-                            .foregroundStyle(Color(.main))
-                    }
+                if let background = Color(hex: tableGroup.color) {
+                    title(backgroundColor: background)
+                } else {
+                    title(backgroundColor: Color(.main))
+                }
 
                 Spacer()
             }
         }
     }
+
+    private func title(backgroundColor: Color) -> some View {
+        Text(tableGroup.name)
+            .font(.title2)
+            .foregroundStyle(backgroundColor.getContentColor(lightColorScheme: .black, darkColorScheme: .white))
+            .padding(6)
+            .background {
+                RoundedRectangle(cornerRadius: 8.0)
+                    .foregroundStyle(backgroundColor)
+            }
+    }
 }
 
 #Preview {

diff --git a/WaiterRobot/Ui/TableList/TableView.swift b/WaiterRobot/Ui/TableList/TableView.swift
@@ -3,26 +3,29 @@ import SwiftUI
 struct TableView: View {
     let text: String
     let hasOrders: Bool
+    let backgroundColor: Color?
     let onClick: () -> Void
 
+    @Environment(\.colorScheme)
+    var colorScheme
+
     var body: some View {
         Button(action: onClick) {
             ZStack {
-                RoundedRectangle(cornerRadius: 20)
-                    .stroke(.blackWhite, lineWidth: 5)
-
                 Text(text)
                     .font(.title)
+                    .frame(maxWidth: .infinity, maxHeight: .infinity)
 
                 if hasOrders {
                     VStack(alignment: .trailing) {
                         HStack {
                             Spacer()
 
                             Circle()
-                                .foregroundColor(.accentColor)
+                                .foregroundColor(backgroundColor?.getContentColor(lightColorScheme: Color(.darkRed), darkColorScheme: Color(.lightRed)))
                                 .frame(width: 12)
                         }
+
                         Spacer()
                     }
                     .padding(.top, 10)
@@ -31,16 +34,25 @@ struct TableView: View {
             }
         }
         .aspectRatio(1.0, contentMode: .fit)
-        .foregroundColor(.blackWhite)
+        .background {
+            if let backgroundColor {
+                RoundedRectangle(cornerRadius: 20)
+                    .foregroundColor(backgroundColor)
+            } else {
+                RoundedRectangle(cornerRadius: 20)
+                    .foregroundColor(.gray.opacity(0.3))
+            }
+        }
+        .foregroundStyle(backgroundColor?.getContentColor(lightColorScheme: .black, darkColorScheme: .white) ?? .blackWhite)
     }
 }
 
 #Preview {
     VStack {
-        TableView(text: "1", hasOrders: false, onClick: {})
+        TableView(text: "1", hasOrders: false, backgroundColor: .green) {}
             .frame(maxWidth: 100)
 
-        TableView(text: "2", hasOrders: true, onClick: {})
+        TableView(text: "2", hasOrders: true, backgroundColor: nil) {}
             .frame(maxWidth: 100)
     }
 }