refactor: restructure DeckCardRowView

Author: dogo

SWDestinyTrades/Classes/SwiftUI/Features/Decks/Components/DeckCardRowView.swift

@@ -30,53 +30,62 @@ struct DeckCardRowView: View {
 
     var body: some View {
         HStack {
-            Image("ic_\(card.typeCode)")
-                .resizable()
-                .renderingMode(.template)
-                .foregroundStyle(card.factionColor())
-                .frame(width: 25, height: 25)
-                .accessibilityHidden(true)
+            cardInfoButton
+            quantityControl
+        }
+        .padding(.vertical, 4)
+        .swipeActions(edge: .trailing, allowsFullSwipe: true) {
+            Button(L10n.delete, role: .destructive) {
+                onRemove()
+            }
+        }
+    }
 
-            Text("\(quantity)")
-                .font(.body)
-                .frame(minWidth: 30)
+    private var cardInfoButton: some View {
+        Button(action: onTap) {
+            HStack {
+                Image("ic_\(card.typeCode)")
+                    .resizable()
+                    .renderingMode(.template)
+                    .foregroundStyle(card.factionColor())
+                    .frame(width: 25, height: 25)
+                    .accessibilityHidden(true)
 
-            VStack(alignment: .leading, spacing: 2) {
-                Text(card.name)
-                    .font(.headline)
-                    .lineLimit(1)
+                Text("\(quantity)")
+                    .font(.body)
+                    .frame(minWidth: 30)
 
-                if !card.subtitle.isEmpty {
-                    Text(card.subtitle)
-                        .font(.caption)
-                        .foregroundStyle(.secondary)
+                VStack(alignment: .leading, spacing: 2) {
+                    Text(card.name)
+                        .font(.headline)
                         .lineLimit(1)
-                }
-            }
-
-            Spacer()
 
-            if card.typeCode == "character", card.isUnique {
-                EliteToggleButton(isElite: $isElite) { newValue in
-                    onEliteToggle(newValue)
+                    if !card.subtitle.isEmpty {
+                        Text(card.subtitle)
+                            .font(.caption)
+                            .foregroundStyle(.secondary)
+                            .lineLimit(1)
+                    }
                 }
-            } else if !(card.typeCode == "character" && card.isUnique), card.typeCode != "battlefield" {
-                Stepper("", value: $quantity, in: 1 ... card.deckLimit) { _ in
-                    onQuantityChange(quantity)
-                }
-                .labelsHidden()
+
+                Spacer()
             }
+            .contentShape(Rectangle())
         }
-        .padding(.vertical, 4)
-        .contentShape(Rectangle())
-        .onTapGesture {
-            onTap()
-        }
-        .accessibilityAddTraits(.isButton)
-        .swipeActions(edge: .trailing, allowsFullSwipe: true) {
-            Button(L10n.delete, role: .destructive) {
-                onRemove()
+        .buttonStyle(.plain)
+    }
+
+    @ViewBuilder
+    private var quantityControl: some View {
+        if card.typeCode == "character", card.isUnique {
+            EliteToggleButton(isElite: $isElite) { newValue in
+                onEliteToggle(newValue)
+            }
+        } else if !(card.typeCode == "character" && card.isUnique), card.typeCode != "battlefield" {
+            Stepper("", value: $quantity, in: 1 ... card.deckLimit) { _ in
+                onQuantityChange(quantity)
             }
+            .labelsHidden()
         }
     }
 }