Closed Bug 1854427 Opened 2 years ago Closed 2 years ago

The header of the review checker sidebar does not remain in-view when scrolled

Categories

(Firefox Graveyard :: Shopping, defect)

Desktop
All
defect

Tracking

(firefox-esr102 unaffected, firefox-esr115 unaffected, firefox117 unaffected, firefox118 disabled, firefox119 verified)

RESOLVED WORKSFORME
Tracking Status
firefox-esr102 --- unaffected
firefox-esr115 --- unaffected
firefox117 --- unaffected
firefox118 --- disabled
firefox119 --- verified

People

(Reporter: pmagyari, Unassigned)

References

(Regression)

Details

(Keywords: regression, Whiteboard: [fidefe-shopping])

Attachments

(1 file)

Found in

  • Firefox 119.0a1 (2023-09-21)

Tested platforms

  • Affected platforms: All

Preconditions

  • browser.shopping.experience2023.enabled- "true"
  • browser.shopping.experience2023.optedIn - "1"

Steps to reproduce

  1. Open Firefox Nightly.
  2. Navigate to an Amazon/Walmart/Bestbuy product page. (example)
  3. Click on the Show more button in the "Highlights from recent reviews" section of the review checker sidebar.
  4. Scroll down inside the review checker sidebar.

Expected result

  • According to the figma spec the header of the sidebar should remain in view regardless of the scroll position.

Actual result

  • The header moves out of view as you scroll down.

Regression range

  • Will return with a regression-range asap.

Mozregression returned the following pushlog: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=c08cba545d2cfff90f54726bd536368b128cfb0b&tochange=a654205d84701ba611f22f0020be612e0db887c6

It seems like Bug 1852302 could be the culprit.

Niklas, could you take a look?
Thanks!

Flags: needinfo?(nbaumgardner)
Keywords: regression
Regressed by: 1852302
Summary: The header of the review checker sidebar does not remain in view when scrolled → The header of the review checker sidebar does not remain in-view when scrolled

I'm surprised that a regressor came up (I don't recall the header ever being in view when scrolling). Nonetheless, yes the patch Gijs linked is for Bug 1851839, which will keep the header stickied to the top.

Flags: needinfo?(nbaumgardner)

:niklas, since you are the author of the regressor, bug 1852302, could you take a look?

For more information, please visit BugBot documentation.

Flags: needinfo?(nbaumgardner)
Flags: needinfo?(nbaumgardner)

I can confirm that this has indeed been fixed by the patch landed for Bug 1851839 so I'm closing this issue.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WORKSFORME
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: