minor changes after code review

facebook · May 4, 2023 · 2cbd5a3 · 2cbd5a3
1 parent 031d5ba
commit 2cbd5a3
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 9 deletions.
diff --git a/packages/react-native/Libraries/Components/TextInput/TextInput.d.ts b/packages/react-native/Libraries/Components/TextInput/TextInput.d.ts
@@ -619,12 +619,14 @@ export interface TextInputProps
   /**
    * Allows to adjust caret height.
    * The default value is 0, which means the height of the caret will be calculated automatically
+   * @platform ios
    */
   caretHeight?: number | undefined;
 
   /**
-   * Allows to adjust caret postiion relative to the Y axis
+   * Allows to adjust caret position relative to the Y axis
    * The default value is 0.
+   * @platform ios
    */
   caretYOffset?: number | undefined;
 

diff --git a/packages/react-native/Libraries/Components/TextInput/TextInput.flow.js b/packages/react-native/Libraries/Components/TextInput/TextInput.flow.js
@@ -550,12 +550,14 @@ export type Props = $ReadOnly<{|
   /**
    * Allows to adjust caret height.
    * The default value is 0, which means the height of the caret will be calculated automatically
+   * @platform ios
    */
-
   caretYOffset?: ?number,
+
   /**
-   * Allows to adjust caret postiion relative to the Y axis
+   * Allows to adjust caret position relative to the Y axis
    * The default value is 0.
+   * @platform ios
    */
   caretYHeight?: ?number,
 

diff --git a/packages/react-native/Libraries/Components/TextInput/TextInput.js b/packages/react-native/Libraries/Components/TextInput/TextInput.js
@@ -339,12 +339,14 @@ type IOSProps = $ReadOnly<{|
   /**
    * Allows to adjust caret height.
    * The default value is 0, which means the height of the caret will be calculated automatically
+   * @platform ios
    */
   caretYOffset?: ?number,
 
   /**
-   * Allows to adjust caret postiion relative to the Y axis
+   * Allows to adjust caret position relative to the Y axis
    * The default value is 0.
+   * @platform ios
    */
   caretHeight?: ?number,
 |}>;
@@ -1459,8 +1461,6 @@ function InternalTextInput(props: Props): React.Node {
         selection={selection}
         style={style}
         text={text}
-        caretYOffset={props.caretYOffset}
-        caretHeight={props.caretHeight}
       />
     );
   } else if (Platform.OS === 'android') {

diff --git a/packages/react-native/Libraries/Text/TextInput/Multiline/RCTUITextView.m b/packages/react-native/Libraries/Text/TextInput/Multiline/RCTUITextView.m
@@ -339,10 +339,11 @@ - (CGRect)caretRectForPosition:(UITextPosition *)position
     originalRect.origin.y += _caretYOffset;
   }
 
-    if(_caretHeight != 0) {
+  if(_caretHeight != 0) {
     originalRect.size.height = _caretHeight;
   }
-    return originalRect;
+
+  return originalRect;
 }
 
 - (NSArray *)selectionRectsForRange:(UITextRange *)range {

diff --git a/...rer/components/textinput/iostextinput/react/renderer/components/iostextinput/primitives.h b/...rer/components/textinput/iostextinput/react/renderer/components/iostextinput/primitives.h
@@ -159,7 +159,6 @@ class TextInputTraits final {
    * iOS-only (inherently iOS-specific)
    * Default value: 0 with a default font.
    */
-
   int caretHeight{0};
 
   /*
-Original file line number
+Diff line change
@@ Expand Up / @@ -159,7 +159,6 @@ class TextInputTraits final { @@
        * iOS-only (inherently iOS-specific)
        * Default value: 0 with a default font.
        */
       int caretHeight{0};
       /*
@@ Expand Down @@