From 39234e862e101fc6b5d4a7a7b9ca4f9ff1ba486f Mon Sep 17 00:00:00 2001 From: Eli White Date: Wed, 11 Dec 2019 14:54:25 -0800 Subject: [PATCH] Fix order of arguments in Fabric measure Summary: These arguments were in the wrong order. It should be x,y,width,height,pageX,pageY. It was x,y,pageX,pageY,width,height. Changelog: [Internal] Reviewed By: yungsters Differential Revision: D18940318 fbshipit-source-id: ebd757648169b1ffbf33b35dded1d505a1c80974 --- ReactCommon/fabric/uimanager/UIManagerBinding.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ReactCommon/fabric/uimanager/UIManagerBinding.cpp b/ReactCommon/fabric/uimanager/UIManagerBinding.cpp index 7aa6156ee05..2825a69d55c 100644 --- a/ReactCommon/fabric/uimanager/UIManagerBinding.cpp +++ b/ReactCommon/fabric/uimanager/UIManagerBinding.cpp @@ -509,10 +509,10 @@ jsi::Value UIManagerBinding::get( runtime, {0, 0, - jsi::Value{runtime, (double)frame.origin.x}, - jsi::Value{runtime, (double)frame.origin.y}, jsi::Value{runtime, (double)frame.size.width}, - jsi::Value{runtime, (double)frame.size.height}}); + jsi::Value{runtime, (double)frame.size.height}, + jsi::Value{runtime, (double)frame.origin.x}, + jsi::Value{runtime, (double)frame.origin.y}}); return jsi::Value::undefined(); }); }