diff options
author | Ollie H <renahlee@outlook.com> | 2023-05-01 18:38:47 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-05-01 20:38:47 -0500 |
commit | 83959c595d52ceb7aa4e3f68441c5ac41c389ebc (patch) | |
tree | 3385d9a16e90fc8d5290ebdef104f922c17642a9 /src/view/com/modals/ChangeHandle.tsx | |
parent | c75c888de2407d3314cad07989174201313facaa (diff) | |
download | voidsky-83959c595d52ceb7aa4e3f68441c5ac41c389ebc.tar.zst |
React Native accessibility (#539)
* React Native accessibility * First round of changes * Latest update * Checkpoint * Wrap up * Lint * Remove unhelpful image hints * Fix navigation * Fix rebase and lint * Mitigate an known issue with the password entry in login * Fix composer dismiss * Remove focus on input elements for web * Remove i and npm * pls work * Remove stray declaration * Regenerate yarn.lock --------- Co-authored-by: Paul Frazee <pfrazee@gmail.com>
Diffstat (limited to 'src/view/com/modals/ChangeHandle.tsx')
-rw-r--r-- | src/view/com/modals/ChangeHandle.tsx | 37 |
1 files changed, 31 insertions, 6 deletions
diff --git a/src/view/com/modals/ChangeHandle.tsx b/src/view/com/modals/ChangeHandle.tsx index 37bad6957..ad7ff5a6d 100644 --- a/src/view/com/modals/ChangeHandle.tsx +++ b/src/view/com/modals/ChangeHandle.tsx @@ -133,7 +133,12 @@ export function Component({onChanged}: {onChanged: () => void}) { <View style={[s.flex1, pal.view]}> <View style={[styles.title, pal.border]}> <View style={styles.titleLeft}> - <TouchableOpacity onPress={onPressCancel}> + <TouchableOpacity + onPress={onPressCancel} + accessibilityRole="button" + accessibilityLabel="Cancel change handle" + accessibilityHint="Exits handle change process" + onAccessibilityEscape={onPressCancel}> <Text type="lg" style={pal.textLight}> Cancel </Text> @@ -148,13 +153,20 @@ export function Component({onChanged}: {onChanged: () => void}) { ) : error && !serviceDescription ? ( <TouchableOpacity testID="retryConnectButton" - onPress={onPressRetryConnect}> + onPress={onPressRetryConnect} + accessibilityRole="button" + accessibilityLabel="Retry change handle" + accessibilityHint={`Retries handle change to ${handle}`}> <Text type="xl-bold" style={[pal.link, s.pr5]}> Retry </Text> </TouchableOpacity> ) : canSave ? ( - <TouchableOpacity onPress={onPressSave}> + <TouchableOpacity + onPress={onPressSave} + accessibilityRole="button" + accessibilityLabel="Save handle change" + accessibilityHint={`Saves handle change to ${handle}`}> <Text type="2xl-medium" style={pal.link}> Save </Text> @@ -245,6 +257,9 @@ function ProvidedHandleForm({ value={handle} onChangeText={onChangeHandle} editable={!isProcessing} + accessible={true} + accessibilityLabel="Handle" + accessibilityHint="Sets Bluesky username" /> </View> <Text type="md" style={[pal.textLight, s.pl10, s.pt10]}> @@ -253,7 +268,11 @@ function ProvidedHandleForm({ @{createFullHandle(handle, userDomain)} </Text> </Text> - <TouchableOpacity onPress={onToggleCustom}> + <TouchableOpacity + onPress={onToggleCustom} + accessibilityRole="button" + accessibilityHint="Hosting provider" + accessibilityLabel="Opens modal for using custom domain"> <Text type="md-medium" style={[pal.link, s.pl10, s.pt5]}> I have my own domain </Text> @@ -338,7 +357,7 @@ function CustomHandleForm({ // = return ( <> - <Text type="md" style={[pal.text, s.pb5, s.pl5]}> + <Text type="md" style={[pal.text, s.pb5, s.pl5]} nativeID="customDomain"> Enter the domain you want to use </Text> <View style={[pal.btn, styles.textInputWrapper]}> @@ -356,6 +375,9 @@ function CustomHandleForm({ value={handle} onChangeText={onChangeHandle} editable={!isProcessing} + accessibilityLabelledBy="customDomain" + accessibilityLabel="Custom domain" + accessibilityHint="Input your preferred hosting provider" /> </View> <View style={styles.spacer} /> @@ -421,7 +443,10 @@ function CustomHandleForm({ )} </Button> <View style={styles.spacer} /> - <TouchableOpacity onPress={onToggleCustom}> + <TouchableOpacity + onPress={onToggleCustom} + accessibilityLabel="Use default provider" + accessibilityHint="Use bsky.social as hosting provider"> <Text type="md-medium" style={[pal.link, s.pl10, s.pt5]}> Nevermind, create a handle for me </Text> |