Closed Bug 1871040 Opened 6 months ago Closed 5 months ago

The Review Checker micro Survey Title is overlapped by the X Close button with DE builds

Categories

(Firefox :: Messaging System, defect, P1)

Desktop
Unspecified
defect

Tracking

()

RESOLVED FIXED
123 Branch
Iteration:
123.2 - Jan 1 - Jan 12
Tracking Status
firefox-esr115 --- disabled
firefox121 --- disabled
firefox122 --- disabled
firefox123 --- verified

People

(Reporter: rdoghi, Assigned: jprickett)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [fidefe-shopping])

Attachments

(3 files)

Attached image 2023-12-20_14h44_02.png

Found in

  • Nightly 123.0a1 (2023-12-20)

Affected versions

  • Nightly 123.0a1 (2023-12-20)
  • Beta 122.0b1

Affected platforms

  • ALL

Preconditions:
Set the browser.shopping.experience2023.enabled - TRUE
Set the browser.shopping.experience2023.optedIn - 1

Have a DE build.

Steps to reproduce

  1. Reach enough Product details pages until the micro Survey is triggered.
  2. Select any option and reach the second slide.
  3. Resize the Review Checker to a smaller size.

Expected result

  • The Text should move to a second row before it reaches the X Close button.

Actual result

  • The X Close button overlaps the text when the Review checker is at a smaller size.

Regression range
N/A

We need to add margin-right: "40px" to the title.

Assignee: nobody → jprickett
Status: NEW → ASSIGNED
Iteration: --- → 123.2 - Jan 1 - Jan 12
Priority: -- → P1
Pushed by jprickett@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0713de5c750a
Add margin to shopping survey header to prevent dismiss overlap r=omc-reviewers,aminomancer
Status: ASSIGNED → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → 123 Branch

Hi @jprickett, the issue underline issue from this bug no longer occurs, the X no longer overlaps the text but with the smallest size the X button is truncated now and a horizontal scrollbar is created, is this ok ?? please check our latest screenshot.

Flags: needinfo?(jprickett)
Attached image Survey.png

I will mark this issue verified as fixed since the main issue here was fixed, and I think the other issue is related to Bug 1870805.

Hello @rdoghi! This is intentional, and is the fix suggested by the product team.

Flags: needinfo?(jprickett)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: