Skip to content

[ACCESSIBILITY - PART 3] Product page improvements#758

Merged
tblivet merged 7 commits into
PrestaShop:improvementsfrom
tblivet:feat/EAA-3
Sep 5, 2025
Merged

[ACCESSIBILITY - PART 3] Product page improvements#758
tblivet merged 7 commits into
PrestaShop:improvementsfrom
tblivet:feat/EAA-3

Conversation

@tblivet

@tblivet tblivet commented Sep 2, 2025

Copy link
Copy Markdown
Contributor
Questions Answers
Description? Product page accessibility improvements
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket? --
Sponsor company @PrestaShopCorp
How to test? --

@tblivet tblivet requested a review from ga-devfront September 2, 2025 15:37
@github-project-automation github-project-automation Bot moved this to Ready for review in PR Dashboard Sep 2, 2025
Comment thread modules/ps_sharebuttons/views/templates/hook/ps_sharebuttons.tpl
Comment thread templates/catalog/_partials/product-variants.tpl
@ps-jarvis

Copy link
Copy Markdown

This pull request seems to contain new translation strings. I have summarized them below to ease up review:

  • Shop.Theme.Global
    • Product availability:
    • Previous image
    • Next image
    • Open zoomed product image gallery
  • Shop.Theme.Actions
    • Remove image
    • Save customization
  • Shop.Theme.Catalog
    • Slide to product image %number%
    • Regular price:
    • Price:
    • :

(Note: this is an automated message, but answering it will reach a real human)

@ps-jarvis ps-jarvis moved this from Ready for review to To be tested in PR Dashboard Sep 3, 2025
@AureRita AureRita self-assigned this Sep 4, 2025

@AureRita AureRita left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @tblivet

Thank you for your PR, I tested it and it seems to works :

  • All interactive elements (zoom icon, swatches, miniatures) are focusable and reachable via keyboard. ✅
  • Focus outlines are clearly visible on all buttons and interactive elements listed. ✅
  • The "Share" elements have meaningful and descriptive accessible names. ✅
  • Screen reader users understand the purpose of the button without ambiguity. ✅

Because the PR seems to works as expected, It's QA ✔️

Thank you

@AureRita AureRita moved this from To be tested to To be merged in PR Dashboard Sep 5, 2025
@tblivet tblivet merged commit db35bc1 into PrestaShop:improvements Sep 5, 2025
6 checks passed
@github-project-automation github-project-automation Bot moved this from To be merged to Merged in PR Dashboard Sep 5, 2025
@ps-jarvis

Copy link
Copy Markdown

PR merged, well done!

Message to @PrestaShop/committers: do not forget to milestone it before the merge.

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

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants