Commit 92d0a735 authored by Robert Knight's avatar Robert Knight

Use `role="status"` for sidebar toolbar status notifier

As suggested in
https://github.com/hypothesis/client/pull/4793#discussion_r972937985, this is an
appropriate role to use for a status display with implied `aria-live="polite"`.
parent 985f1acb
......@@ -46,7 +46,7 @@ function ToolbarButton({ ...buttonProps }) {
*/
function StatusNotifier({ highlightsVisible }) {
return (
<div className="sr-only" aria-live="polite" data-testid="toolbar-status">
<div className="sr-only" role="status" data-testid="toolbar-status">
{highlightsVisible ? 'Highlights visible' : 'Highlights hidden'}
</div>
);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment