Closed Bug 1915092 Opened 2 years ago Closed 1 year ago

[Menu Redesign] Menu CFR is incorrectly positioned after rotating screen

Categories

(Firefox for Android :: Toolbar, defect, P2)

All
Android
defect

Tracking

()

RESOLVED FIXED
134 Branch
Tracking Status
firefox134 --- fixed

People

(Reporter: petru, Assigned: tchoh)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fxdroid][group3])

Attachments

(3 files, 1 external file)

Attached video MenuCFRAnchoring.mp4 —

Steps to reproduce

  1. Ensure the redesigned menu has not been shown before
  2. Open the redesigned menu
  3. While the redesigned menu CFR is shown rotate the screen

Expected behavior

The CFR is still shown at the top of the menu and in the horizontal middle

Actual behavior

The CFR is placed in the left bottom half of the menu

Assignee: nobody → tchoh
Status: NEW → ASSIGNED
Whiteboard: [fxdroid][group3]

https://www.figma.com/design/RFz9fYtotQCQuinwcZujZt/%5B%5D-Menu-Redesign?node-id=11412-108977&t=qUJn1l57c0a2dlgy-4

Following a discussion with the design team, the requirements have changed for displaying the CFR. The CFR will only be shown the first time the Menu is opened for an existing user. If the device is rotated, the CFR will be dismissed and will not reappear.

We'll want to fix this CFR bug before our beta experiment.

Priority: P3 → P2

This is the behavior on a tablet.
The arrow is not sticked on the CFR, as it can be seen in the short video.

Tested with Samsung Galaxy Tab S9 Ultra (Android 14), and Lenovo TB X606X (Android 10).

Pushed by tchoh@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9efa2d7e9e4c Menu CFR orientation position change r=android-reviewers,skhan
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 134 Branch

This is the behavior on the latest Fenix Nightly 134.0a1 from 11/20 on a Samsung Galaxy Tab S9 Ultra (Android 14) tablet.

Flags: needinfo?(petru)

@Tim think that for at least one other CFR we decided with UX to automatically dismiss it when the screen is rotated? Should we try something similar here?

Flags: needinfo?(petru) → needinfo?(tchoh)
Regressions: 1932553
See Also: → 1940679
Flags: needinfo?(tchoh)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: