diff --git a/example/src/Examples.tsx b/example/src/Examples.tsx index cd955552..6a02f786 100644 --- a/example/src/Examples.tsx +++ b/example/src/Examples.tsx @@ -182,7 +182,6 @@ export const examples = [ }, { title: 'Disabled slider', - platform: 'android', render() { return ; }, diff --git a/package/android/src/main/java/com/reactnativecommunity/slider/ReactSliderManagerImpl.java b/package/android/src/main/java/com/reactnativecommunity/slider/ReactSliderManagerImpl.java index f59e9bb2..f07beb5d 100644 --- a/package/android/src/main/java/com/reactnativecommunity/slider/ReactSliderManagerImpl.java +++ b/package/android/src/main/java/com/reactnativecommunity/slider/ReactSliderManagerImpl.java @@ -56,8 +56,8 @@ public static void setStep(ReactSlider view, float value) { view.setStep(value); } - public static void setEnabled(ReactSlider view, boolean enabled) { - view.setEnabled(enabled); + public static void setDisabled(ReactSlider view, boolean disabled) { + view.setEnabled(!disabled); } public static void setThumbTintColor(ReactSlider view, Integer color) { diff --git a/package/android/src/newarch/java/com/reactnativecommunity/slider/ReactSliderManager.java b/package/android/src/newarch/java/com/reactnativecommunity/slider/ReactSliderManager.java index e82cd08e..ff522858 100644 --- a/package/android/src/newarch/java/com/reactnativecommunity/slider/ReactSliderManager.java +++ b/package/android/src/newarch/java/com/reactnativecommunity/slider/ReactSliderManager.java @@ -89,9 +89,9 @@ protected ReactSlider createViewInstance(ThemedReactContext context) { } @Override - @ReactProp(name = ViewProps.ENABLED, defaultBoolean = true) - public void setEnabled(ReactSlider view, boolean enabled) { - ReactSliderManagerImpl.setEnabled(view, enabled); + @ReactProp(name = "disabled", defaultBoolean = false) + public void setDisabled(ReactSlider view, boolean disabled) { + ReactSliderManagerImpl.setDisabled(view, disabled); } @Override @@ -176,10 +176,6 @@ public Map getExportedCustomDirectEventTypeConstants() { } // these props are not available on Android, however we must override their setters - @Override - @ReactProp(name = "disabled") - public void setDisabled(ReactSlider view, boolean disabled) {} - @Override public void setMinimumTrackImage(ReactSlider view, @Nullable ReadableMap readableMap) {} diff --git a/package/android/src/oldarch/java/com/reactnativecommunity/slider/ReactSliderManager.java b/package/android/src/oldarch/java/com/reactnativecommunity/slider/ReactSliderManager.java index 83c4a5ff..af1ed1e0 100644 --- a/package/android/src/oldarch/java/com/reactnativecommunity/slider/ReactSliderManager.java +++ b/package/android/src/oldarch/java/com/reactnativecommunity/slider/ReactSliderManager.java @@ -116,9 +116,9 @@ protected ReactSlider createViewInstance(ThemedReactContext context) { return ReactSliderManagerImpl.createViewInstance(context); } - @ReactProp(name = ViewProps.ENABLED, defaultBoolean = true) - public void setEnabled(ReactSlider view, boolean enabled) { - ReactSliderManagerImpl.setEnabled(view, enabled); + @ReactProp(name = "disabled", defaultBoolean = false) + public void setDisabled(ReactSlider view, boolean disabled) { + ReactSliderManagerImpl.setDisabled(view, disabled); } @ReactProp(name = "value", defaultFloat = 0f) diff --git a/package/src/RNCSliderNativeComponent.ts b/package/src/RNCSliderNativeComponent.ts index 3d3e28a1..5708106c 100644 --- a/package/src/RNCSliderNativeComponent.ts +++ b/package/src/RNCSliderNativeComponent.ts @@ -18,7 +18,6 @@ export interface NativeProps extends ViewProps { accessibilityUnits?: string; accessibilityIncrements?: ReadonlyArray; disabled?: WithDefault; - enabled?: WithDefault; inverted?: WithDefault; vertical?: WithDefault; tapToSeek?: WithDefault; diff --git a/package/src/RNCSliderNativeComponent.web.tsx b/package/src/RNCSliderNativeComponent.web.tsx index fbf21400..4d877937 100644 --- a/package/src/RNCSliderNativeComponent.web.tsx +++ b/package/src/RNCSliderNativeComponent.web.tsx @@ -32,7 +32,7 @@ export interface Props { thumbStyle: ViewStyle; style: ViewStyle; inverted: boolean; - enabled: boolean; + disabled: boolean; trackHeight: number; thumbSize: number; onRNCSliderSlidingStart: (value: number) => void; @@ -53,7 +53,7 @@ const RCTSliderWebComponent = React.forwardRef( thumbStyle = {}, style = {}, inverted = false, - enabled = true, + disabled = false, trackHeight = 4, thumbSize = 20, onRNCSliderSlidingStart = (_: number) => {}, @@ -273,8 +273,8 @@ const RCTSliderWebComponent = React.forwardRef( accessible={true} accessibilityRole={'adjustable'} style={containerStyle} - onStartShouldSetResponder={() => enabled} - onMoveShouldSetResponder={() => enabled} + onStartShouldSetResponder={() => !disabled} + onMoveShouldSetResponder={() => !disabled} onResponderGrant={() => onSlidingStart(value)} onResponderRelease={onTouchEnd} onResponderMove={onMove} diff --git a/package/src/Slider.tsx b/package/src/Slider.tsx index 66b63c1d..e3039b9b 100644 --- a/package/src/Slider.tsx +++ b/package/src/Slider.tsx @@ -238,7 +238,6 @@ const SliderComponent = ( onRNCSliderSlidingStart={onSlidingStartEvent} onRNCSliderSlidingComplete={onSlidingCompleteEvent} onRNCSliderValueChange={onValueChangeEvent} - enabled={!_disabled} disabled={_disabled} onStartShouldSetResponder={() => true} onResponderTerminationRequest={() => false} diff --git a/package/src/__tests__/__snapshots__/Slider.test.tsx.snap b/package/src/__tests__/__snapshots__/Slider.test.tsx.snap index 4c0c44b8..aec986b4 100644 --- a/package/src/__tests__/__snapshots__/Slider.test.tsx.snap +++ b/package/src/__tests__/__snapshots__/Slider.test.tsx.snap @@ -8,7 +8,6 @@ exports[` accessibilityState disabled sets disabled={true} 1`] = ` } } disabled={true} - enabled={false} inverted={false} maximumValue={1} minimumValue={0} @@ -38,7 +37,6 @@ exports[` disabled prop overrides accessibilityState.disabled 1`] = ` } } disabled={true} - enabled={false} inverted={false} maximumValue={1} minimumValue={0} @@ -68,7 +66,6 @@ exports[` disabled prop overrides accessibilityState.enabled 1`] = ` } } disabled={false} - enabled={true} inverted={false} maximumValue={1} minimumValue={0} @@ -93,7 +90,6 @@ exports[` disabled prop overrides accessibilityState.enabled 1`] = ` exports[` renders a slider with custom props 1`] = ` renders disabled slider 1`] = ` } } disabled={true} - enabled={false} inverted={false} maximumValue={1} minimumValue={0} @@ -151,7 +146,6 @@ exports[` renders disabled slider 1`] = ` exports[` renders enabled slider 1`] = `