fix: solve share bottomsheet issue

uwaisalqadri · Jan 29, 2025 · 65d6cd8 · 65d6cd8
1 parent e446a3d
commit 65d6cd8
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 13 deletions.
diff --git a/Giffy/Features/Home/Button.swift b/Giffy/Features/Home/Button.swift
@@ -19,8 +19,8 @@ struct IconButton: View {
       Image(systemName: iconName)
         .font(.system(size: size))
         .foregroundColor(tint)
-        .frame(width: size, height: size)
         .contentShape(.rect)
+        .padding(8)
     }
     .buttonStyle(.plain)
   }

diff --git a/Giffy/Features/Sticker/StickerReducer.swift b/Giffy/Features/Sticker/StickerReducer.swift
@@ -53,8 +53,7 @@ public struct StickerReducer {
     case presentPhotoPicker(_ state: Bool)
     case toggleShowOriginalImage(_ state: Bool)
     case updateSticker(_ sticker: Sticker)
-    case shareSticker
-    case dismissShare
+    case shouldShowShare(_ state: Bool)
   }
 
   public var body: some Reducer<State, Action> {
@@ -85,12 +84,8 @@ public struct StickerReducer {
         state.currentSticker = sticker
         return .none
 
-      case .shareSticker:
-        state.isCopied = true
-        return .none
-
-      case .dismissShare:
-        state.isCopied = false
+      case let .shouldShowShare(bool):
+        state.isCopied = bool
         return .none
       }
     }

diff --git a/Giffy/Features/Sticker/StickerView.swift b/Giffy/Features/Sticker/StickerView.swift
@@ -64,13 +64,13 @@ struct StickerView: View {
         shouldDismissOnTapOutside: true,
         isShowing: viewStore.binding(
           get: { $0.isCopied },
-          send: .dismissShare
+          send: { .shouldShowShare($0) }
         )
       ) {
         ShareView(store: viewStore.state.share)
       }
-      .onChange(of: viewStore.state.currentSticker) { sticker, _ in
-        tabState.isShowShare = sticker.imageData != nil
+      .onChange(of: viewStore.isCopied) { _, _ in
+        tabState.isShowShare = viewStore.isCopied
       }
       .navigationBarTitleDisplayMode(.inline)
       .toolbar {
@@ -86,7 +86,7 @@ struct StickerView: View {
               tint: .Theme.green,
               size: 15,
               onClick: {
-                viewStore.send(.shareSticker)
+                viewStore.send(.shouldShowShare(true))
               }
             )
             .tapScaleEffect()