Merge pull request #342 from edx/fix/post-button

[iOS] "Post" button is almost invisible in active state for Discussions
openedx · Mar 19, 2024 · d704613 · d704613
2 parents fd3fa4c + 96106dc
commit d704613
Showing 1 changed file with 12 additions and 5 deletions.
diff --git a/Core/Core/View/Base/FlexibleKeyboardInputView.swift b/Core/Core/View/Base/FlexibleKeyboardInputView.swift
@@ -24,6 +24,10 @@ public struct FlexibleKeyboardInputView: View {
         self.sendText = sendText
     }
 
+    private var canSend: Bool {
+        commentText.trimmingCharacters(in: .whitespacesAndNewlines).count > 0
+    }
+
     public var body: some View {
         VStack {
             VStack {
@@ -71,19 +75,22 @@ public struct FlexibleKeyboardInputView: View {
                                         )
                                 ).padding(8)
                             Button(action: {
-                                if commentText.trimmingCharacters(in: .whitespacesAndNewlines).count > 0 {
+                                if canSend {
                                     sendText(commentText)
                                     self.commentText = ""
                                 }
                             }, label: {
                                 VStack {
-                                    commentText.trimmingCharacters(in: .whitespacesAndNewlines).count > 0
-                                    ? CoreAssets.send.swiftUIImage
-                                    : CoreAssets.sendDisabled.swiftUIImage
+                                    CoreAssets.send.swiftUIImage
+                                        .renderingMode(.template)
+                                        .foregroundStyle(Theme.Colors.accentXColor)
+                                        .opacity(canSend ? 1 : 0.5)
                                 }
                                 .frame(width: 36, height: 36)
                                 .foregroundColor(Theme.Colors.white)
-                            }).padding(.top, 8)
+                            })
+                            .padding(.top, 8)
+                            .disabled(!canSend)
 
                         }.padding(.horizontal, isHorizontal ? 50 : 16)