Closed Bug 1865822 Opened 1 year ago Closed 1 year ago

The Review Checker will show a horizontal scrollbar for an unanalyzed product

Categories

(Firefox :: Shopping, defect)

Desktop
Unspecified
defect

Tracking

()

VERIFIED FIXED
122 Branch
Tracking Status
firefox-esr115 --- disabled
firefox120 --- disabled
firefox121 --- wontfix
firefox122 --- verified

People

(Reporter: rdoghi, Assigned: kpatenio)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fidefe-shopping])

Attachments

(2 files)

Attached image 2023-11-21_15h04_44.png

Found in

  • Nightly 122.0a1 (2023-11-21)

Affected versions

  • Nightly 122.0a1 (2023-11-21)
  • Beta 121

Affected platforms

  • ALL

Preconditions:
browser.shopping.experience2023.enabled - true
browser.shopping.experience2023.optedIn = 0

Steps to reproduce

  1. Reach https://www.walmart.com/ip/Gain-Ultra-Dish-Washing-Liquid-Dish-Soap-Honey-Berry-Hula-Scent-90-fl-oz/1683982875
  2. Resize the Review checker sidebar to a smaller size.

Expected result

  • The horizontal Scrollbar should not be displayed.

Actual result

  • A horizontal scrollbar is displayed when the unanalyzed card is displayed.

Regression range
Not Applicable

:rdoghi, if you think that's a regression, could you try to find a regression range using for example mozregression?

From quick testing, I'm pretty sure Bug 1865808 is the culprit. The unanalyzed product card image spilling out will cause horizontal overflow on the sidebar, hence why there is the horizontal scrollbar. I don't know what's a good size for the image though with the smallest sidebar width, so we'll need UX input for that.

See Also: → 1865808
Whiteboard: [fidefe-shopping]
Assignee: nobody → kpatenio
Status: NEW → ASSIGNED
See Also: → 1865351
Pushed by kpatenio@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6d55e68bd862 adjust unanalyzed product card image for smaller sidebar width. r=shopping-reviewers,Gijs
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 122 Branch

The patch landed in nightly and beta is affected.
:kpatenio, is this bug important enough to require an uplift?

  • If yes, please nominate the patch for beta approval.
  • If no, please set status-firefox121 to wontfix.

For more information, please visit BugBot documentation.

Flags: needinfo?(kpatenio)

Verified as fixed in our latest Nightly build 122.0a1 (2023-11-28) , It looks GREAT !

Flags: needinfo?(kpatenio)

Updating the main status flag.

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: