Skip to content

Commit 33d149a

Browse files
authored
Fix/TextField - Refactored TextField accessibility props logic (#730)
* Refactored TextField accessibility props logic * Cosmetics * Removing redundant condition
1 parent c75f581 commit 33d149a

File tree

1 file changed

+5
-8
lines changed

1 file changed

+5
-8
lines changed

src/components/inputs/TextField.js

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -225,21 +225,18 @@ export default class TextField extends BaseInput {
225225
}
226226

227227
getAccessibilityInfo() {
228-
const {floatingPlaceholder, placeholder, expandable, value} = this.getThemeProps();
228+
const {floatingPlaceholder, placeholder} = this.getThemeProps();
229+
const accessibilityState = this.isDisabled() ? {disabled: true} : undefined;
230+
let accessibilityLabel = floatingPlaceholder ? this.props.accessibilityLabel || placeholder : '';
229231

230-
let accessibilityLabel = floatingPlaceholder ? placeholder : undefined;
231232
if (this.isRequiredField()) {
232-
accessibilityLabel = `${accessibilityLabel || ''}. Mandatory`;
233-
}
234-
if (expandable) {
235-
accessibilityLabel = `${accessibilityLabel || ''}. ${value || ''}`;
233+
accessibilityLabel = `${accessibilityLabel}. Mandatory`;
236234
}
237235

238-
const accessibilityStates = this.isDisabled() ? ['disabled'] : [];
239236
return {
240237
accessibilityLabel,
241238
// on Android accessibilityStates cause issues with expandable input
242-
accessibilityStates: Constants.isIOS ? accessibilityStates : undefined
239+
accessibilityState: Constants.isIOS ? accessibilityState : undefined
243240
};
244241
}
245242

0 commit comments

Comments
 (0)