Skip to content

Commit

Permalink
offline notice: Queue non-urgent a11y announcements behind in-progress
Browse files Browse the repository at this point in the history
ones
fixes: #5611
  • Loading branch information
1010nishant committed Jan 25, 2023
1 parent 966c36a commit de56346
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions src/boot/OfflineNoticeProvider.js
Original file line number Diff line number Diff line change
Expand Up @@ -212,9 +212,10 @@ export function OfflineNoticeProvider(props: ProviderProps): Node {
// another app.)

if (shouldShowUncertaintyNotice && !haveAnnouncedUncertain.current) {
// TODO(react-native-68): Use announceForAccessibilityWithOptions to
// queue this behind any in-progress announcements
AccessibilityInfo.announceForAccessibility(_('Zulip’s Internet connection is uncertain.'));
AccessibilityInfo.announceForAccessibilityWithOptions(
_('Zulip’s Internet connection is uncertain.'),
{ queue: true },
);
haveAnnouncedUncertain.current = true;
}

Expand All @@ -226,9 +227,7 @@ export function OfflineNoticeProvider(props: ProviderProps): Node {
isOnline === true
&& (haveAnnouncedOffline.current || haveAnnouncedUncertain.current)
) {
// TODO(react-native-68): Use announceForAccessibilityWithOptions to
// queue this behind any in-progress announcements
AccessibilityInfo.announceForAccessibility(_('Zulip is online.'));
AccessibilityInfo.announceForAccessibilityWithOptions(_('Zulip is online.'), { queue: true });
haveAnnouncedOffline.current = false;
haveAnnouncedUncertain.current = false;
}
Expand Down

0 comments on commit de56346

Please sign in to comment.