1873660 - Voice over will read How we determine review Quality and Settings 3x after opting in

Status: NEW

(Firefox :: Shopping, defect)

Description

[Rares Doghi, Desktop QA]

Found in

• Beta 122.0b06

Affected versions

• Nightly 123.0a1 (2023-01-11)
• Beta 122.0b06

Affected platforms

• macOS

Preconditions:
Set the browser.shopping.experience2023.enabled - TRUE
Set the browser.shopping.experience2023.optedIn - 0
Enable Voice Over

Enable Voice Over

Steps to reproduce

1. Reach any Product Details page that has never been analyzed.
2. Use voice over commands in order to reach the Yes Try it! button.
3. Select the Yes Try it! button with VO Space.
4. Wait for the Voice over to read out loud the entire Sidebar.

Expected result

• The How we determine review quality and Settings titles should only be read out loud once.

Actual result

• The How we determine Review Quality and Settings titles are read 3 times.

Regression range
39:55.26 INFO: Last good revision: 7da2e7486803b79322d275486f0b74b860249546
39:55.26 INFO: First bad revision: af897d54469b6ee7a6b902fe9e3c5ee72bb54fc2
39:55.26 INFO: Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=7da2e7486803b79322d275486f0b74b860249546&tochange=af897d54469b6ee7a6b902fe9e3c5ee72bb54fc2

Comment 1

[Donal Meehan [:dmeehan]]

:niklas, since you are the author of the regressor, bug 1860585, could you take a look? Also, could you confirm the severity is S3?

Comment 2

[Anna Yeddi [:ayeddi]]

the opt-in part of the checker has role of alert which is then dynamically removed and may be causing issues for a screen reader AND/OR the header wrapper for H1 is not necessary since it is a small component that only includes up to 2 short elements one of which is a heading anyway (vs. when the <header> is used to contain the entire complex header of an e-commerce website with multiple menus and controls) - either of these may be the cause of VoiceOver repeating itself, I think.

Setting is up as an access-s3 since it is not a blocker, but does increase verbosity that would likely affect if a screen reader user would know how to use the Review Checker itself right after the opt-in.

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1860585