From c7aa3954e347352b9e800cb9ec3b4e9800366250 Mon Sep 17 00:00:00 2001 From: Saad Najmi Date: Wed, 4 Oct 2023 06:42:17 -0700 Subject: [PATCH] Move `[RCTConvert RCTTransformOrigin:]` out of UIAccessibilityTraits category (#39758) Summary: https://github.com/facebook/react-native/commit/5f40f0800e64b4380d7897b2e8b9ff561d84b97c added a new prop and corresponding RCTConvert method for transformOrigin. Strangely, it added the RCTConvertMethod to a `RCTConvert+UIAccessibilityTraits` category.. which feels like the wrong spot for it. Let's just add it to the existing `RCTConvert+Transform` category. This also means we can get rid of the header if we move the `RCTTransformOrigin` struct into UIView+React (where it is used). ## Changelog: [IOS][CHANGED] - Move `[RCTConvert RCTTransformOrigin:]` out of UIAccessibilityTraits category Pull Request resolved: https://github.com/facebook/react-native/pull/39758 Test Plan: Transform Origin example still works. Screenshot 2023-10-02 at 11 26 17 AM Reviewed By: javache Differential Revision: D49867993 Pulled By: NickGerleman fbshipit-source-id: 1b2d5a9d08f0231040e7449f2eb75860f08bafa5 --- .../React/Views/RCTConvert+Transform.h | 3 +++ .../React/Views/RCTConvert+Transform.m | 7 +++++++ .../react-native/React/Views/RCTTransformOrigin.h | 15 --------------- .../react-native/React/Views/RCTViewManager.m | 8 -------- packages/react-native/React/Views/UIView+React.h | 8 +++++++- 5 files changed, 17 insertions(+), 24 deletions(-) delete mode 100644 packages/react-native/React/Views/RCTTransformOrigin.h diff --git a/packages/react-native/React/Views/RCTConvert+Transform.h b/packages/react-native/React/Views/RCTConvert+Transform.h index 9ad72aef37..728e85fa79 100644 --- a/packages/react-native/React/Views/RCTConvert+Transform.h +++ b/packages/react-native/React/Views/RCTConvert+Transform.h @@ -6,9 +6,12 @@ */ #import "RCTConvert.h" +#import "UIView+React.h" @interface RCTConvert (Transform) + (CATransform3D)CATransform3D:(id)json; ++ (RCTTransformOrigin)RCTTransformOrigin:(id)json; + @end diff --git a/packages/react-native/React/Views/RCTConvert+Transform.m b/packages/react-native/React/Views/RCTConvert+Transform.m index 348d4ec4ab..8248b5892a 100644 --- a/packages/react-native/React/Views/RCTConvert+Transform.m +++ b/packages/react-native/React/Views/RCTConvert+Transform.m @@ -144,4 +144,11 @@ static const NSUInteger kMatrixArrayLength = 4 * 4; return transform; } ++ (RCTTransformOrigin)RCTTransformOrigin:(id)json +{ + RCTTransformOrigin transformOrigin = { + [RCTConvert YGValue:json[0]], [RCTConvert YGValue:json[1]], [RCTConvert CGFloat:json[2]]}; + return transformOrigin; +} + @end diff --git a/packages/react-native/React/Views/RCTTransformOrigin.h b/packages/react-native/React/Views/RCTTransformOrigin.h deleted file mode 100644 index 1823fd91f1..0000000000 --- a/packages/react-native/React/Views/RCTTransformOrigin.h +++ /dev/null @@ -1,15 +0,0 @@ -/* - * Copyright (c) Meta Platforms, Inc. and affiliates. - * - * This source code is licensed under the MIT license found in the - * LICENSE file in the root directory of this source tree. - */ - -#import -#import - -typedef struct { - YGValue x; - YGValue y; - CGFloat z; -} RCTTransformOrigin; diff --git a/packages/react-native/React/Views/RCTViewManager.m b/packages/react-native/React/Views/RCTViewManager.m index 9b4775ceec..5f5c6ce190 100644 --- a/packages/react-native/React/Views/RCTViewManager.m +++ b/packages/react-native/React/Views/RCTViewManager.m @@ -15,7 +15,6 @@ #import "RCTConvert.h" #import "RCTLog.h" #import "RCTShadowView.h" -#import "RCTTransformOrigin.h" #import "RCTUIManager.h" #import "RCTUIManagerUtils.h" #import "RCTUtils.h" @@ -122,13 +121,6 @@ RCT_MULTI_ENUM_CONVERTER( UIAccessibilityTraitNone, unsignedLongLongValue) -+ (RCTTransformOrigin)RCTTransformOrigin:(id)json -{ - RCTTransformOrigin transformOrigin = { - [RCTConvert YGValue:json[0]], [RCTConvert YGValue:json[1]], [RCTConvert CGFloat:json[2]]}; - return transformOrigin; -} - @end @implementation RCTViewManager diff --git a/packages/react-native/React/Views/UIView+React.h b/packages/react-native/React/Views/UIView+React.h index 46bd1660aa..fbc8ca0f94 100644 --- a/packages/react-native/React/Views/UIView+React.h +++ b/packages/react-native/React/Views/UIView+React.h @@ -8,11 +8,17 @@ #import #import -#import +#import #import @class RCTShadowView; +typedef struct { + YGValue x; + YGValue y; + CGFloat z; +} RCTTransformOrigin; + @interface UIView (React) /**