a11y NPM Resource Changed PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar AchalShrivastava
AchalShrivastava
10 Sep 2026

Pull Request resolves #48429.

  • 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

  • Increased the target size of the "Need Support?" links on the Joomla Administrator login page.
  • Set the links to a 24px line-height and inline-block display.
  • This provides a 24px target height for the support links.

Testing Instructions

  1. Open the Joomla Administrator login page.
  2. Navigate to the "Need Support?" section.
  3. Inspect the Support Forum, Documentation, and News links.
  4. Verify that each link has a 24px height.
  5. Verify that the links remain visually aligned and usable.

Actual result BEFORE applying this Pull Request

The "Need Support?" links had a target height of approximately 16.5px, which did not meet the 24px minimum target size requirement.

Expected result AFTER applying this Pull Request

The "Need Support?" links have a 24px target height, improving their target size in accordance with WCAG 2.2 SC 2.5.8.

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

avatar AchalShrivastava AchalShrivastava - open - 10 Sep 2026
avatar AchalShrivastava AchalShrivastava - change - 10 Sep 2026
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 Sep 2026
Category ⇒ Repository NPM Change
avatar AchalShrivastava AchalShrivastava - change - 10 Sep 2026
Labels Added: NPM Resource Changed PR-5.4-dev
avatar brianteeman
brianteeman - comment - 10 Sep 2026

Shouldnt this be more specific and on the .loginsupport and not the entire wrapper?

avatar AchalShrivastava
AchalShrivastava - comment - 11 Sep 2026

Shouldnt this be more specific and on the .loginsupport and not the entire wrapper?

It's doing its work properly as mentioned in WCAG 2.2 SC 2.5.8 24px target height is there or u can explain me properly as I'm a beginner

avatar brianteeman
brianteeman - comment - 11 Sep 2026

This targets everything and anything that could be placed in the sidebar. It must be restricted to only the items you want to fix.

avatar AchalShrivastava
AchalShrivastava - comment - 12 Sep 2026

This targets everything and anything that could be placed in the sidebar. It must be restricted to only the items you want to fix.

Now I have done necessary changes pls review it.

avatar mbeganyi-a11y mbeganyi-a11y - test_item - 21 Sep 2026 - Tested successfully
avatar mbeganyi-a11y
mbeganyi-a11y - comment - 21 Sep 2026

I have tested this item ✅ successfully on 97d6939

I notice a shift in the entire login section compared to other standard login screens (without this PR). The entire section seems to be shifted down by some small number of pixels. I think this could be applied with more care only to the "Need Support?" section without impacting the login section. Is that fair? I marked this as successfully tested, but if this is merged, it has consequences for the UI that maintainers may take issue with. I report a successful test only as a humble tester, as the issue it is trying to resolve is indeed resolved, but with consequences. More investigation should be done.


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

avatar mbeganyi-a11y
mbeganyi-a11y - comment - 21 Sep 2026

I have tested this item ✅ successfully on 97d6939

I notice a shift in the entire login section compared to other standard login screens (without this PR). The entire section seems to be shifted down by some small number of pixels. I think this could be applied with more care only to the "Need Support?" section without impacting the login section. Is that fair? I marked this as successfully tested, but if this is merged, it has consequences for the UI that maintainers may take issue with. I report a successful test only as a humble tester, as the issue it is trying to resolve is indeed resolved, but with consequences. More investigation should be done.


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

avatar mbeganyi-a11y
mbeganyi-a11y - comment - 21 Sep 2026

I have attached two images of equal dimension demonstrating this shift. One image shows the original login screen and the other the login screen after this PR. For the sake of the issue it's trying to solve, it is solved, hence my "tested successfully" choice. Please download and compare side-by-side, as this vertical layout makes it difficult to compare:

Shifted login section down by some pixels after applying this PR Standard login section without this PR
avatar mbeganyi-a11y
mbeganyi-a11y - comment - 29 Sep 2026

Is anyone available to test and also decide about whether the shifting in layout is acceptable? If not, happy to help out to close this PR.

avatar rdeutz
rdeutz - comment - 5 Oct 2026

I would target the change so that it only affects the module.

avatar mbeganyi-a11y
mbeganyi-a11y - comment - 6 Oct 2026

I've created a new PR to target only the .loginsupport module. If that is successfully implemented, this PR can be closed.

avatar rdeutz rdeutz - close - 6 Oct 2026
avatar rdeutz rdeutz - change - 6 Oct 2026
Status Pending ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2026-10-06 16:39:35
Closed_By ⇒ rdeutz
Labels Added: a11y
avatar rdeutz
rdeutz - comment - 6 Oct 2026

closed in favor of #48582

Add a Comment

Login with GitHub to post a comment