Skip to content

Improve checkbox and radio accessibility - #248

Merged
Quetzacoalt91 merged 1 commit into
PrestaShop:developfrom
tblivet:fix/custom-checkbox
Nov 25, 2025
Merged

Quetzacoalt91 merged 1 commit into
PrestaShop:developfrom
tblivet:fix/custom-checkbox

Conversation

@tblivet

@tblivet tblivet commented Nov 17, 2025

Copy link
Copy Markdown
Contributor
Questions Answers
Description? Improve checkbox and radio accessibility
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket? Related PR/Issue from the core PrestaShop/PrestaShop#40011
Sponsor company @PrestaShopCorp
How to test? Run the storybook of the project and check if "Stylised checkboxes" & "Radio buttons" works using keyboard.

After the fixe using keyboard:

a11y-improvement-checkbox-radio.mov

@github-project-automation github-project-automation Bot moved this to Ready for review in PR Dashboard Nov 17, 2025
@ChillCode

ChillCode commented Nov 17, 2025

Copy link
Copy Markdown

Had time to test this UI thing and got this result after applied your patch, BO Invoices:

Tested with Firefox:

checkbox.mp4

For example the same applies to all grids using checkboxes for ID, like products and categories.

Thanks!

@Quetzacoalt91

Copy link
Copy Markdown
Member

Just tried the branch on the login page as reported by the issue.

In the folder [admin]/themes/new-theme, run:

npm install --save tblivet/prestashop-ui-kit_fork#fix/custom-checkbox

Then compile the assets.

Capture.video.du.2025-11-20.17-38-01.mp4

@PrestaShop/committers Is there anything more needed before we can merge this PR?

@Quetzacoalt91 Quetzacoalt91 added this to the 2.0.4 milestone Nov 25, 2025
@Quetzacoalt91
Quetzacoalt91 merged commit 16b6d4d into PrestaShop:develop Nov 25, 2025
1 check passed
@github-project-automation github-project-automation Bot moved this from Ready for review to Merged in PR Dashboard Nov 25, 2025
@ps-jarvis ps-jarvis moved this from Merged to Ready for review in PR Dashboard Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants