A11y: Use DOM element for booking form title.#548
A11y: Use DOM element for booking form title.#548peterwilsoncc wants to merge 3 commits intotrunkfrom
Conversation
|
This is linked to GitHub issue #549. |
|
This is linked to GitHub issue #580. |
|
@qasumitbagthariya Can you give this PR a test when you get a chance? |
qasumitbagthariya
left a comment
There was a problem hiding this comment.
QA Update ✅
I have verified this PR in the fix/WOOACBK-70-a11y-booking-form branch, which has been fixed and is functioning as intended.
I tested the following on this branch:
Untitled.mov
Testing Environment
Details
- WordPress: 6.9.4
- Theme: Twenty Twenty-Five 1.4
- WooCommerce - 10.6.2
- PHP: 8.0.30
- Web Server: Nginx 1.20.2
- Browser: Chrome
- OS: macOS 15.2
- Branch: fix/WOOACBK-70-a11y-booking-form
Steps to Test- As mentioned in the PR description.
Test Results - It is working as expected.
Functional Demo / Screencast -
Special Notes - Ready for UAT
Testing Document status:
Cases related to this Issue/PR are added to the Critical Flow Wiki pages:
- Yes
- Not Required/Applicable for this PR
Regression / Smoke Test Report ✅Tested with Archive File created via "php woorelease.phar build repo_URL" (Composer version 2.8.4, npm version 10.8.2, node version 20.18.1) Status- Working expected with Plugin Archive/Zip file same as fix specific branch. Testing Environment Details
Next Step- Ready to Merge 🚀 |
All Submissions:
Changes proposed in this Pull Request:
Closes https://linear.app/a8c/issue/WOOA11Y-436/dynamic-text-not-announced-or-reachable-with-screen-readers-wooext-329
Steps to test the changes in this Pull Request:
Changelog entry
Closes #580