Skip to content

Small fixes related to the global Hummingbird theme QA review#832

Merged
tblivet merged 4 commits intoPrestaShop:developfrom
tblivet:fix/qa-review
Nov 25, 2025
Merged

Small fixes related to the global Hummingbird theme QA review#832
tblivet merged 4 commits intoPrestaShop:developfrom
tblivet:fix/qa-review

Conversation

@tblivet
Copy link
Contributor

@tblivet tblivet commented Nov 24, 2025

Questions Answers
Description? Small fixes related to the global Hummingbird theme QA review
Type? bug fix
BC breaks? no
Deprecations? no
Fixed ticket? --
Sponsor company @PrestaShopCorp
How to test? --

@tblivet tblivet requested a review from ga-devfront November 24, 2025 15:46
@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Nov 24, 2025
ga-devfront
ga-devfront previously approved these changes Nov 24, 2025
@ps-jarvis ps-jarvis moved this from Ready for review to To be tested in PR Dashboard Nov 24, 2025
@AureRita AureRita self-assigned this Nov 24, 2025
Copy link

@AureRita AureRita left a comment

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 correct all the divers issue seen, as you can see :

Capture.video.du.2025-11-25.12-12-13.mp4
Capture d’écran du 2025-11-24 18-11-57 Capture d’écran du 2025-11-24 18-25-34

and the screen reader correctly read the part for the number of product contain in your cart

Tested on :
9.0.1-classic
9.1.0 (develop

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

Thank you

@tblivet tblivet merged commit 67e0eda into PrestaShop:develop Nov 25, 2025
6 checks passed
@github-project-automation github-project-automation bot moved this from To be tested to Merged in PR Dashboard Nov 25, 2025
@ps-jarvis
Copy link

PR merged, well done!

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

@tblivet tblivet added this to the v2.0.0 milestone Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants