BigBlueButton/bigbluebutton-Github
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(screen-reader): add safeguards for adding alerts

Browse Source 
Adds checks for the alert's text and DOM element before adding screen
reader alerts.
This commit is contained in:
Arthurk12 2024-04-11 14:34:41 -03:00 committed by prlanzarin
parent fcbfcb1bbc
commit 6cb0e914ab
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
bigbluebutton-html5/imports/ui/components/screenreader-alert/component.jsx
View File

@ -9,7 +9,10 @@ const ScreenReaderAlert = ({ olderAlert }) => {
if (olderAlert) setTimeout(() => removeAlert(olderAlert.id), ARIA_ALERT_EXT_TIMEOUT);
}, [olderAlert?.id]);
return olderAlert ? createPortal(olderAlert.text, document.getElementById('aria-polite-alert')) : null;
const ariaAlertsElement = document.getElementById('aria-polite-alert');
const shouldAddAlert = olderAlert && olderAlert.text && ariaAlertsElement !== null;
return shouldAddAlert ? createPortal(olderAlert.text, ariaAlertsElement) : null;
};
export default ScreenReaderAlert;