diff options
author | Paul Frazee <pfrazee@gmail.com> | 2023-07-03 15:57:53 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-07-03 15:57:53 -0500 |
commit | bc55241c9ae731f633f8b93a9b7eac7635070148 (patch) | |
tree | 2f75cf5fc31bb8214f848a4326c1af731000b842 /src/view/com/util/Link.tsx | |
parent | 0163ba0af8cd42e2a76b3ae66efc0777afdc2544 (diff) | |
download | voidsky-bc55241c9ae731f633f8b93a9b7eac7635070148.tar.zst |
[APP-724] Collection of accessibility fixes (#949)
* Fix: include alt text on the web lightbox image * a11y: Dont read the 'ALT' label * a11y: remove a wrapper behavior from posts This appears to have been introduced with the goal of creating meta actions on posts, but the behavior seems counter-productive. The accessibility inspector was unable to access individual items within the post and therefore most content was simply skipped. There may be a way to support the post actions without losing the ability to access the inner elements but I couldnt find it. -prf * a11y: apply alt tags to image wrappers so they get read * a11y: set Link accessibilityLabel to the title if none set * a11y: skip the SANDBOX watermark * a11y: improve post meta to not read UI and give a useful date * ally: improve post controls * a11y: add labels to lightbox images on mobile * fix types
Diffstat (limited to 'src/view/com/util/Link.tsx')
-rw-r--r-- | src/view/com/util/Link.tsx | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/src/view/com/util/Link.tsx b/src/view/com/util/Link.tsx index f753f01cc..7ff896344 100644 --- a/src/view/com/util/Link.tsx +++ b/src/view/com/util/Link.tsx @@ -88,6 +88,10 @@ export const Link = observer(function Link({ props.dataSet.noUnderline = 1 } + if (title && !props.accessibilityLabel) { + props.accessibilityLabel = title + } + return ( <TouchableOpacity testID={testID} @@ -171,6 +175,7 @@ export const DesktopWebTextLink = observer(function DesktopWebTextLink({ text, numberOfLines, lineHeight, + ...props }: { testID?: string type?: TypographyVariant @@ -179,6 +184,9 @@ export const DesktopWebTextLink = observer(function DesktopWebTextLink({ text: string | JSX.Element numberOfLines?: number lineHeight?: number + accessible?: boolean + accessibilityLabel?: string + accessibilityHint?: string }) { if (isDesktopWeb) { return ( @@ -190,6 +198,7 @@ export const DesktopWebTextLink = observer(function DesktopWebTextLink({ text={text} numberOfLines={numberOfLines} lineHeight={lineHeight} + {...props} /> ) } @@ -199,7 +208,8 @@ export const DesktopWebTextLink = observer(function DesktopWebTextLink({ type={type} style={style} numberOfLines={numberOfLines} - lineHeight={lineHeight}> + lineHeight={lineHeight} + {...props}> {text} </Text> ) |