Open Bug 1826740 Opened 2 years ago Updated 11 months ago

Certain CFRs no longer point to the correct position when changing the orientation

Categories

(Fenix :: Tabs, defect)

Firefox 113
All
Android
defect

Tracking

(firefox112 affected, firefox113 affected, firefox123 affected, firefox124 affected, firefox125 affected)

ASSIGNED
Tracking Status
firefox112 --- affected
firefox113 --- affected
firefox123 --- affected
firefox124 --- affected
firefox125 --- affected

People

(Reporter: dpop, Assigned: vdreghici)

References

(Depends on 1 open bug, Blocks 3 open bugs)

Details

(Whiteboard: [fxdroid])

Attachments

(2 files)

Attached video Inactive Tabs CFR.mp4

Steps to reproduce

  1. Launch Firefox for Android and open a few tabs.
  2. Have one tab selected and send the application in background.
  3. Go to device's settings and set the time to 14 days in the future.
  4. Bring Firefox for Android foreground and access the Tabs Tray.
  5. Once the Inactive Tabs CFR is displayed, change the orientation.
  6. Observe the Inactive Tabs CFR.

Expected behavior

The Inactive Tabs CFR should still point to the inactive tabs section when the orientation is changed.

Actual behavior

The Inactive Tabs CFR no longer points to the inactive tabs section when changing the orientation is changed, regardless of the initial orientation.

Device information

  • Firefox version: Nightly 113.0a1, Beta 112.0b9, RC 112.0
  • Android device model: Lenovo Yoga Tab 11 (and any device)
  • Android OS version: Android 11

Any additional information?

Summary: The Inactive Tabs CFR no longer points to the inactive tabs section when changing the orientation is changed → The Inactive Tabs CFR no longer points to the inactive tabs section when changing the orientation

Indexing in Bug 1818308 just so we don't lose track of this one.

Whiteboard: [fxdroid]

I have looked into this issue a bit and noticed that inactive tabs CFR is actually an Android dialog. Since there have been already established a set of behaviors for all CFRs in the app, and we have started working on them, migrating private browsing recommendation CFR already to CFRPopup, I suppose it is safe to say that we could do the same with this CFR, in order to unify CFRs behaviors in the app.

Assignee: nobody → Vlad.DreghiciPopa
Depends on: 1812461
Depends on: 1855942
See Also: → 1855951
Status: NEW → ASSIGNED
Blocks: 1862372

I'm combining this bug with https://bugzilla.mozilla.org/show_bug.cgi?id=1855951 for simplicity.

Two examples of orientation changes causing issues with CFRs:

  • The shopping CFRs is not pointing to the shopping icon when turning in landscape mode
  • The Inactive Tabs CFR no longer points to the inactive tabs section when changing the orientation

These should probably be fixed in the same PR.

Summary: The Inactive Tabs CFR no longer points to the inactive tabs section when changing the orientation → Certain CFRs no longer point to the correct position when changing the orientation
Duplicate of this bug: 1855951
Blocks: 1857438

The severity field for this bug is set to S4. However, the following bug duplicate has higher severity:

:vdreghici, could you consider increasing the severity of this bug to S3?

For more information, please visit BugBot documentation.

Flags: needinfo?(Vlad.DreghiciPopa)

Leaving S4 for now as it is a minor cosmetic issue

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

Attachment

General

Created:
Updated:
Size: