(iOS) Fix that the vertical paddings are 0. It should be 8 by default

Author: mshibanami

Sources/ResizingTextView/ResizingTextView.swift

@@ -138,6 +138,11 @@ public struct ResizingTextView: View, Equatable {
             .opacity(isFocused && isEditable ? 1 : 0).scaleEffect(isFocused && isEditable ? 1 : 1.03))
 #elseif os(iOS)
         ZStack(alignment: .topLeading) {
+            
+            /// HACK: In iOS 17, the last sentence of a non-editable text may not be drawn if the textContainerInset is `.zero`. To avoid it, we add this 0.00...1 value to the insets.
+            let defaultTextContainerInset = UIEdgeInsets(top: 8, left: 0.00000001, bottom: 8, right: 0.00000001)
+            let effectiveTextContainerInset = textContainerInset ?? defaultTextContainerInset
+            
             TextView(
                 $text,
                 isEditable: isEditable,
@@ -148,18 +153,18 @@ public struct ResizingTextView: View, Equatable {
                 canHaveNewLineCharacters: canHaveNewLineCharacters,
                 foregroundColor: Color(foregroundColor),
                 autocapitalizationType: autocapitalizationType,
-                textContainerInset: textContainerInset
+                textContainerInset: effectiveTextContainerInset
             )
             if let placeholder {
                 let isLTR = layoutDirection == .leftToRight
                 Text(placeholder)
                     .font(Font(font))
                     .lineLimit(1)
                     .foregroundColor(Color(foregroundColor.withAlphaComponent(0.2)))
-                    .padding(.top, textContainerInset?.top ?? 0)
-                    .padding(isLTR ? .leading : .trailing, textContainerInset?.left ?? 0)
-                    .padding(.bottom, textContainerInset?.bottom ?? 0)
-                    .padding(isLTR ? .trailing : .leading, textContainerInset?.right ?? 0)
+                    .padding(.top, effectiveTextContainerInset.top)
+                    .padding(isLTR ? .leading : .trailing, effectiveTextContainerInset.left)
+                    .padding(.bottom, effectiveTextContainerInset.bottom)
+                    .padding(isLTR ? .trailing : .leading, effectiveTextContainerInset.right)
                     .allowsHitTesting(false)
                     .opacity(text.isEmpty ? 1 : 0)
             }

Sources/ResizingTextView/TextView (UIKit).swift

@@ -7,9 +7,6 @@ import UIKit
 struct TextView: UIViewRepresentable {
     static let defaultForegroundColor = Color(UIColor.label)
 
-    /// HACK: In iOS 17, the last sentence of a non-editable text may not be drawn if the textContainerInset is `.zero`. To avoid it, we add this default value to the insets.
-    private static let defaultTextContainerInset = UIEdgeInsets(top: 0.00000001, left: 0.00000001, bottom: 0.00000001, right: 0.00000001)
-
     @Binding private var text: String
     private var isEditable: Bool
     private var isScrollable: Bool
@@ -20,7 +17,7 @@ struct TextView: UIViewRepresentable {
     private var canHaveNewLineCharacters: Bool
     private var width: CGFloat?
     private var autocapitalizationType: UITextAutocapitalizationType
-    private var textContainerInset: UIEdgeInsets?
+    private var textContainerInset: UIEdgeInsets
 
     init(
         _ text: Binding<String>,
@@ -32,7 +29,7 @@ struct TextView: UIViewRepresentable {
         canHaveNewLineCharacters: Bool,
         foregroundColor: Color,
         autocapitalizationType: UITextAutocapitalizationType,
-        textContainerInset: UIEdgeInsets?
+        textContainerInset: UIEdgeInsets
     ) {
         _text = text
         self.isEditable = isEditable
@@ -84,7 +81,7 @@ struct TextView: UIViewRepresentable {
         if view.autocapitalizationType != autocapitalizationType {
             view.autocapitalizationType = autocapitalizationType
         }
-        let textContainerInset = textContainerInset ?? Self.defaultTextContainerInset
+        let textContainerInset = textContainerInset
         if view.textContainerInset != textContainerInset {
             view.textContainerInset = textContainerInset
         }