From afbe9272cb9bd33989999c8bf721bc03891d6dab Mon Sep 17 00:00:00 2001 From: Sharon Gong Date: Mon, 7 Oct 2019 15:27:56 -0700 Subject: [PATCH] Add accessibilityValue prop on Touchables --- Libraries/Components/Touchable/TouchableBounce.js | 1 + Libraries/Components/Touchable/TouchableHighlight.js | 1 + .../Touchable/TouchableNativeFeedback.android.js | 1 + Libraries/Components/Touchable/TouchableOpacity.js | 1 + .../Components/Touchable/TouchableWithoutFeedback.js | 4 ++++ .../js/examples/Accessibility/AccessibilityExample.js | 8 +++++--- 6 files changed, 13 insertions(+), 3 deletions(-) diff --git a/Libraries/Components/Touchable/TouchableBounce.js b/Libraries/Components/Touchable/TouchableBounce.js index a2cc708c43580c..68bf5b7b7999de 100644 --- a/Libraries/Components/Touchable/TouchableBounce.js +++ b/Libraries/Components/Touchable/TouchableBounce.js @@ -183,6 +183,7 @@ const TouchableBounce = ((createReactClass({ accessibilityState={this.props.accessibilityState} accessibilityActions={this.props.accessibilityActions} onAccessibilityAction={this.props.onAccessibilityAction} + accessibilityValue={this.props.accessibilityValue} nativeID={this.props.nativeID} testID={this.props.testID} hitSlop={this.props.hitSlop} diff --git a/Libraries/Components/Touchable/TouchableHighlight.js b/Libraries/Components/Touchable/TouchableHighlight.js index dac77da436cfc1..08d436a373f12d 100644 --- a/Libraries/Components/Touchable/TouchableHighlight.js +++ b/Libraries/Components/Touchable/TouchableHighlight.js @@ -408,6 +408,7 @@ const TouchableHighlight = ((createReactClass({ accessibilityHint={this.props.accessibilityHint} accessibilityRole={this.props.accessibilityRole} accessibilityState={this.props.accessibilityState} + accessibilityValue={this.props.accessibilityValue} accessibilityActions={this.props.accessibilityActions} onAccessibilityAction={this.props.onAccessibilityAction} style={StyleSheet.compose( diff --git a/Libraries/Components/Touchable/TouchableNativeFeedback.android.js b/Libraries/Components/Touchable/TouchableNativeFeedback.android.js index 83231387f81220..e9ef048c452b01 100644 --- a/Libraries/Components/Touchable/TouchableNativeFeedback.android.js +++ b/Libraries/Components/Touchable/TouchableNativeFeedback.android.js @@ -316,6 +316,7 @@ const TouchableNativeFeedback = createReactClass({ accessibilityState: this.props.accessibilityState, accessibilityActions: this.props.accessibilityActions, onAccessibilityAction: this.props.onAccessibilityAction, + accessibilityValue: this.props.accessibilityValue, children, testID: this.props.testID, onLayout: this.props.onLayout, diff --git a/Libraries/Components/Touchable/TouchableOpacity.js b/Libraries/Components/Touchable/TouchableOpacity.js index 8f70ff1231d00e..3be1844e1553ce 100644 --- a/Libraries/Components/Touchable/TouchableOpacity.js +++ b/Libraries/Components/Touchable/TouchableOpacity.js @@ -313,6 +313,7 @@ const TouchableOpacity = ((createReactClass({ accessibilityState={this.props.accessibilityState} accessibilityActions={this.props.accessibilityActions} onAccessibilityAction={this.props.onAccessibilityAction} + accessibilityValue={this.props.accessibilityValue} style={[this.props.style, {opacity: this.state.anim}]} nativeID={this.props.nativeID} testID={this.props.testID} diff --git a/Libraries/Components/Touchable/TouchableWithoutFeedback.js b/Libraries/Components/Touchable/TouchableWithoutFeedback.js index f00abb22de058c..adc2422898930e 100755 --- a/Libraries/Components/Touchable/TouchableWithoutFeedback.js +++ b/Libraries/Components/Touchable/TouchableWithoutFeedback.js @@ -34,6 +34,7 @@ import type { AccessibilityState, AccessibilityActionInfo, AccessibilityActionEvent, + AccessibilityValue, } from '../View/ViewAccessibility'; type TargetEvent = SyntheticEvent< @@ -55,6 +56,7 @@ const OVERRIDE_PROPS = [ 'accessibilityState', 'accessibilityActions', 'onAccessibilityAction', + 'accessibilityValue', 'hitSlop', 'nativeID', 'onBlur', @@ -71,6 +73,7 @@ export type Props = $ReadOnly<{| accessibilityRole?: ?AccessibilityRole, accessibilityState?: ?AccessibilityState, accessibilityActions?: ?$ReadOnlyArray, + accessibilityValue?: ?AccessibilityValue, children?: ?React.Node, delayLongPress?: ?number, delayPressIn?: ?number, @@ -112,6 +115,7 @@ const TouchableWithoutFeedback = ((createReactClass({ accessibilityState: PropTypes.object, accessibilityActions: PropTypes.array, onAccessibilityAction: PropTypes.func, + accessibilityValue: PropTypes.object, /** * When `accessible` is true (which is the default) this may be called when * the OS-specific concept of "focus" occurs. Some platforms may not have diff --git a/RNTester/js/examples/Accessibility/AccessibilityExample.js b/RNTester/js/examples/Accessibility/AccessibilityExample.js index e910fcf02c560b..c2fc33a0ae4eb5 100644 --- a/RNTester/js/examples/Accessibility/AccessibilityExample.js +++ b/RNTester/js/examples/Accessibility/AccessibilityExample.js @@ -564,7 +564,7 @@ class FakeSliderExample extends React.Component { }}> Fake Slider - - Equalizer - + + Equalizer + + ); }