From 5ac848cd7f90f55447d9022fec8d864cd032fc93 Mon Sep 17 00:00:00 2001 From: Janic Duplessis Date: Sat, 16 Jan 2016 02:32:39 -0800 Subject: [PATCH] Allow setting elevation on any type of View Summary: Moved setElevation from ReactViewManager to BaseViewManager. This allows setting elevation on any type of view such as Toolbar for exemple. Closes https://github.com/facebook/react-native/pull/5292 Reviewed By: svcscm Differential Revision: D2837415 Pulled By: androidtrunkagent fb-gh-sync-id: a5aa7abd95b99303d312cd9c2414539335540925 --- .../com/facebook/react/uimanager/BaseViewManager.java | 9 +++++++++ .../com/facebook/react/views/view/ReactViewManager.java | 8 -------- 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java index 26e8adaad0b1ba..9a697ce3a5417d 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java +++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java @@ -26,6 +26,7 @@ public abstract class BaseViewManager= Build.VERSION_CODES.LOLLIPOP) { + view.setElevation(PixelUtil.toPixelFromDIP(elevation)); + } + // Do nothing on API < 21 + } @ReactProp(name = PROP_RENDER_TO_HARDWARE_TEXTURE) public void setRenderToHardwareTexture(T view, boolean useHWTexture) { diff --git a/ReactAndroid/src/main/java/com/facebook/react/views/view/ReactViewManager.java b/ReactAndroid/src/main/java/com/facebook/react/views/view/ReactViewManager.java index 0f3973bd2bb06e..5e761b0f7a2e58 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/views/view/ReactViewManager.java +++ b/ReactAndroid/src/main/java/com/facebook/react/views/view/ReactViewManager.java @@ -61,14 +61,6 @@ public void setBorderStyle(ReactViewGroup view, @Nullable String borderStyle) { view.setBorderStyle(borderStyle); } - @ReactProp(name = "elevation") - public void setElevation(ReactViewGroup view, float elevation) { - if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) { - view.setElevation(PixelUtil.toPixelFromDIP(elevation)); - } - // Do nothing on API < 21 - } - @ReactProp(name = "pointerEvents") public void setPointerEvents(ReactViewGroup view, @Nullable String pointerEventsStr) { if (pointerEventsStr != null) {