From a07abe718826d62302f2573f18e74fb2b5433677 Mon Sep 17 00:00:00 2001 From: Emil Sjolander Date: Tue, 15 Nov 2016 18:52:09 -0800 Subject: [PATCH] Correctly translate nowrap -> NO_WRAP Summary: The java enum was recently changed from NOWRAP -> NO_WRAP so the translation from js failed. This fixes that. Reviewed By: limichaelc Differential Revision: D4186869 fbshipit-source-id: fe35211a6632d80356d35a01a079279ef4bd7006 --- .../com/facebook/react/uimanager/LayoutShadowNode.java | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/LayoutShadowNode.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/LayoutShadowNode.java index ce88dd5c49f..06e039eb7a9 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/LayoutShadowNode.java +++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/LayoutShadowNode.java @@ -86,8 +86,13 @@ public class LayoutShadowNode extends ReactShadowNode { @ReactProp(name = ViewProps.FLEX_WRAP) public void setFlexWrap(@Nullable String flexWrap) { - setFlexWrap( - flexWrap == null ? CSSWrap.NO_WRAP : CSSWrap.valueOf(flexWrap.toUpperCase(Locale.US))); + if (flexWrap == null || flexWrap.equals("nowrap")) { + setFlexWrap(CSSWrap.NO_WRAP); + } else if (flexWrap.equals("wrap")) { + setFlexWrap(CSSWrap.WRAP); + } else { + throw new IllegalArgumentException("Unknown flexWrap value: " + flexWrap); + } } @ReactProp(name = ViewProps.ALIGN_SELF)