starred/radzen-blazor
1
0
Fork 0
mirror of https://github.com/radzenhq/radzen-blazor.git synced 2026-02-04 05:35:44 +00:00
Code Issues 1.3k Packages Projects Releases 22 Wiki Activity

[PR #1571] [MERGED] Changes aria-live regions for better screen reader #2816

New Issue
Open
opened 2026-01-29 18:20:36 +00:00 by claunia · 0 comments
claunia commented 2026-01-29 18:20:36 +00:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/radzenhq/radzen-blazor/pull/1571
Author: @efinder2
Created: 6/25/2024
Status: ✅ Merged
Merged: 6/27/2024
Merged by: @enchev

Base: master ← Head: aria-live

📝 Commits (3)

📊 Changes

3 files changed (+3 additions, -3 deletions)

View changed files

📝 Radzen.Blazor/RadzenNotification.razor (+1 -1)
📝 Radzen.Blazor/RadzenNotificationMessage.razor (+1 -1)
📝 RadzenBlazorDemos/Shared/EventConsole.razor (+1 -1)

📄 Description

According to the documentation of the mdn live regions are only prompted if they are present before the content changes / is added.

The live region of the RadzenAlert was moved outside of the if(visible). If notifications are added, they will be announced.

For the same reason the aria-live was moved from the NotificationMessage to the Notification.

Also the role=log was added to the example console so changes were announced there.

Those changes were tested with NVDA.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/radzenhq/radzen-blazor/pull/1571 **Author:** [@efinder2](https://github.com/efinder2) **Created:** 6/25/2024 **Status:** ✅ Merged **Merged:** 6/27/2024 **Merged by:** [@enchev](https://github.com/enchev) **Base:** `master` ← **Head:** `aria-live` --- ### 📝 Commits (3) - [`a950f55`](https://github.com/radzenhq/radzen-blazor/commit/a950f5599907a7cfd95837099372b9ec50a6fc66) changed aria for screen reader - [`df96051`](https://github.com/radzenhq/radzen-blazor/commit/df96051cc77af72d57396abb426db7c647da2214) fix div - [`b22ce92`](https://github.com/radzenhq/radzen-blazor/commit/b22ce92c4e3a405d26875ae05ddaec1c16cbd617) reverted RadzenAlert ### 📊 Changes **3 files changed** (+3 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `Radzen.Blazor/RadzenNotification.razor` (+1 -1) 📝 `Radzen.Blazor/RadzenNotificationMessage.razor` (+1 -1) 📝 `RadzenBlazorDemos/Shared/EventConsole.razor` (+1 -1) </details> ### 📄 Description According to the documentation of the [mdn ](https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/ARIA_Live_Regions) live regions are only prompted if they are present **before** the content changes / is added. The live region of the RadzenAlert was moved outside of the `if(visible)`. If notifications are added, they will be announced. For the same reason the aria-live was moved from the NotificationMessage to the Notification. Also the `role=log` was added to the example console so changes were announced there. Those changes were tested with NVDA. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-29 18:20:36 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
enhancement
good first issue
pull-request
Mirrored from GitHub Pull Request
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
claunia
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/radzen-blazor#2816
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?