From 3af01a8a44853246d3deb957c4a1c65e43848bb5 Mon Sep 17 00:00:00 2001 From: Nick Gerleman Date: Wed, 3 Jul 2024 10:05:13 -0700 Subject: [PATCH] Fix default for `showsHorizontalScrollIndicator` and `showsVerticalScrollIndicator` (#45258) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/45258 Setting this to false, then removing, will not reset back to default state of `true`. Add explicit defaults so that the absence of the prop will lead to scroll indicators always being shown. Changelog: [Android][Fixed] - Fix default for `showsHorizontalScrollIndicator` and `showsVerticalScrollIndicator` Reviewed By: javache Differential Revision: D59285745 fbshipit-source-id: 6a7c204cfe9c4ab9e4efbbda300cdfdaf57e8f37 --- .../react/views/scroll/ReactHorizontalScrollViewManager.java | 2 +- .../com/facebook/react/views/scroll/ReactScrollViewManager.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/scroll/ReactHorizontalScrollViewManager.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/scroll/ReactHorizontalScrollViewManager.java index 86abb6baacb..fa3ceb951bf 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/scroll/ReactHorizontalScrollViewManager.java +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/scroll/ReactHorizontalScrollViewManager.java @@ -80,7 +80,7 @@ public class ReactHorizontalScrollViewManager extends ViewGroupManager view.setFocusable(value); } - @ReactProp(name = "showsVerticalScrollIndicator") + @ReactProp(name = "showsVerticalScrollIndicator", defaultBoolean = true) public void setShowsVerticalScrollIndicator(ReactScrollView view, boolean value) { view.setVerticalScrollBarEnabled(value); }