Skip to content

[5.4]Language Installation Info [a11y]#47653

Open
brianteeman wants to merge 2 commits intojoomla:5.4-devfrom
brianteeman:language_install_a11y
Open

[5.4]Language Installation Info [a11y]#47653
brianteeman wants to merge 2 commits intojoomla:5.4-devfrom
brianteeman:language_install_a11y

Conversation

@brianteeman
Copy link
Copy Markdown
Contributor

Pull Request resolves # .

  • I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.

Summary of Changes

Removes the aria-hidden on the info icon and replaces it with an aria-label this serves a similar purpose to hiding the icon as it replaces the icon with the aria-label
This element can receive keyboard focus, but is hidden from screen readers by an aria-hidden="true" attribute which is an accessibility error

Testing Instructions

Check the list of available languages that can be installed and check that any outdated language has an info icon which can be tabbed to and display a tooltip and/or the tooltip is displayed on hover

Actual result BEFORE applying this Pull Request

Accessibility testing reports an error

Expected result AFTER applying this Pull Request

No change in behaviour but accessibility testing does not report an error

Link to documentations

Please select:

  • Documentation link for guide.joomla.org:

  • No documentation changes for guide.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

Signed-off-by: BrianTeeman <brian@teeman.net>
@krishnagandhicode
Copy link
Copy Markdown
Contributor

krishnagandhicode commented Apr 24, 2026

I have tested this item ✅ successfully on cfd9843

Tested successfully during PR Testing meet (24-04-2026)


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

@ThomasFinnern
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on cfd9843

Checked the console in devTools and the eror is gone


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

@adarshdubey03
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on cfd9843

Tested Successfully!


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

@richard67
Copy link
Copy Markdown
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Apr 24, 2026
@richard67 richard67 added the a11y Accessibility label Apr 24, 2026
@richard67 richard67 added this to the Joomla! 5.4.6 milestone Apr 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a11y Accessibility PR-5.4-dev RTC This Pull Request is Ready To Commit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants