RTC a11y NPM Resource Changed bug PR-5.3-dev Pending

User tests: Successful: Unsuccessful:

avatar Fedik
Fedik
13 Sep 2025

Pull Request for Issue #45429

Summary of Changes

Add support for aria-label and aria-labelledby attributes for dialog in the JoomlaDialog.
The attributes can be set manually during dialog creation.
When it is empty the attribute will be set automatically based on the header text.

Testing Instructions

Apply patch.
Run npm install.
Open any dialog in backend and check source code for it.

Actual result BEFORE applying this Pull Request

Dialog does not have aria-label attribute

Expected result AFTER applying this Pull Request

Dialog has an aria-label attribute.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:
  • No documentation changes for docs.joomla.org needed
  • Pull Request link for manual.joomla.org: TBD
  • No documentation changes for manual.joomla.org needed
avatar Fedik Fedik - open - 13 Sep 2025
avatar Fedik Fedik - change - 13 Sep 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 13 Sep 2025
Category JavaScript Repository NPM Change
avatar Fedik Fedik - change - 13 Sep 2025
Title
[5.4] Joomla Dialog add support for aria-label
[5.3] Joomla Dialog add support for aria-label
avatar Fedik Fedik - edited - 13 Sep 2025
avatar ceford ceford - test_item - 18 Sep 2025 - Tested successfully
avatar ceford
ceford - comment - 18 Sep 2025

I have tested this item ✅ successfully on 8894fd9

With the patch, the Batch process dialog has this:

<dialog aria-label="Batch process the selected articles" style="width: 100%; max-width: 800px; height: fit-content;" open="">

Without the patch the aria-label is absent.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46090.

avatar alikon alikon - test_item - 18 Sep 2025 - Tested successfully
avatar alikon
alikon - comment - 18 Sep 2025

I have tested this item ✅ successfully on 8894fd9


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46090.

avatar alikon alikon - change - 18 Sep 2025
Status Pending Ready to Commit
avatar alikon
alikon - comment - 18 Sep 2025

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46090.

avatar brianteeman brianteeman - test_item - 18 Sep 2025 - Tested successfully
avatar brianteeman
brianteeman - comment - 18 Sep 2025

I have tested this item ✅ successfully on 8894fd9


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46090.

avatar LadySolveig LadySolveig - change - 20 Sep 2025
Labels Added: RTC a11y NPM Resource Changed bug PR-5.3-dev
avatar LadySolveig LadySolveig - close - 20 Sep 2025
avatar LadySolveig LadySolveig - merge - 20 Sep 2025
avatar LadySolveig LadySolveig - change - 20 Sep 2025
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2025-09-20 08:20:25
Closed_By LadySolveig
avatar LadySolveig
LadySolveig - comment - 20 Sep 2025

Thank you for the fix @Fedik great teamwork - bringing up and clarifying the issue and also for testing @ceford @alikon @brianteeman @chmst @bembelimen

Add a Comment

Login with GitHub to post a comment