Skip to content

Comments

[5.1] Cog icon focus [a11y]#43135

Merged
wilsonge merged 2 commits intojoomla:5.1-devfrom
brianteeman:focus
Mar 29, 2024
Merged

[5.1] Cog icon focus [a11y]#43135
wilsonge merged 2 commits intojoomla:5.1-devfrom
brianteeman:focus

Conversation

@brianteeman
Copy link
Contributor

adds an outline to the cog when it receives keyboard focus

Pull Request for Issue #43066 .

Expected result AFTER applying this Pull Request

image

image

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:

  • No documentation changes for manual.joomla.org needed

adds an outline to the cog when it receives keyboard focus
@Quy
Copy link
Contributor

Quy commented Mar 24, 2024

I have tested this item ✅ successfully on 9bcda96


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

1 similar comment
@viocassel
Copy link
Contributor

I have tested this item ✅ successfully on 9bcda96


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

@Quy
Copy link
Contributor

Quy commented Mar 24, 2024

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Mar 24, 2024
@Quy Quy added the a11y Accessibility label Mar 28, 2024
@wilsonge wilsonge enabled auto-merge (squash) March 29, 2024 22:21
@wilsonge wilsonge added this to the Joomla! 5.1.0 milestone Mar 29, 2024
@wilsonge wilsonge merged commit 8a3e608 into joomla:5.1-dev Mar 29, 2024
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Mar 29, 2024
@brianteeman
Copy link
Contributor Author

thanks @wilsonge

@brianteeman brianteeman deleted the focus branch March 29, 2024 23:13
@wilsonge
Copy link
Contributor

Thanks!

@bembelimen
Copy link
Contributor

bembelimen commented Mar 29, 2024

This PR is not optimal, because the selector is too generic. Now all buttons in the cpanel which are the first child get this effect. It should be either reverted or fixed (by being more specific).

@brianteeman
Copy link
Contributor Author

can you provide an example please so I can check

@brianteeman
Copy link
Contributor Author

From what I can see the only button that can ever match the selector is the cog - what am I missing?

@bembelimen
Copy link
Contributor

It selected every item with the class "btn" which is the first child. So if a module has a btn, it gets the styling, too.

@chmst
Copy link
Contributor

chmst commented Apr 1, 2024

My concern is that a second button (for a custom module) does not get the focus-ring

@brianteeman
Copy link
Contributor Author

brianteeman commented Apr 1, 2024

It selected every item with the class "btn" which is the first child. So if a module has a btn, it gets the styling, too.

demonstrably not true, otherwise the buttons in the sample data module would have the outline

My concern is that a second button (for a custom module) does not get the focus-ring

Please show an example

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a11y Accessibility NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants