Closed Bug 1912697 Opened 1 year ago Closed 11 months ago

Refresh gets broken on HistoryFragment when device theme has changed

Categories

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

Firefox 125
All
Android
defect

Tracking

()

VERIFIED FIXED
135 Branch
Tracking Status
firefox129 --- wontfix
firefox132 --- wontfix
firefox133 --- wontfix
firefox134 --- wontfix
firefox135 --- verified

People

(Reporter: exploratory666, Assigned: giorga)

References

(Depends on 1 open bug, Regression)

Details

(Keywords: regression, Whiteboard: [fxdroid][group4])

Attachments

(2 files)

Attached video Screen_recording.mp4

Steps to reproduce

  1. Click the '...' button
  2. Click the 'History' button
  3. Go to device settings
  4. Change device theme (does not matter if it is to dark or to light)
  5. Come back to 'Firefox'
  6. Click the 'Search' icon
  7. Swipe down the screen for updating
  8. Click the 'Delete' button on the top right of the screen

Expected behavior

The 'Search' icon should respond by allowing a search to be initiated.
The 'Refresh' icon should briefly display and then update the content.
The 'Delete' icon can work normally

Actual behavior

In step 6, 'Search' icon has no response
In step 7, 'Refresh' icon shows forever
In step 8, the 'Delete' button is clickable but fails to completely delete the history.

see also

See Also: → 1907779
Status: UNCONFIRMED → NEW
Ever confirmed: true

I can reproduce this error as well.

Flags: needinfo?(cpeterson)
Severity: -- → S3
Whiteboard: [fxdroid][group4]
Flags: needinfo?(cpeterson)

Changing the title to only reflect the issue with refresh as the other issues already have bugs filed for them

Summary: After changing the device theme, 'Search' icon is unresponsive, 'Refresh' icon spins indefinitely and 'Delete' button works abnormaly on 'History' page → Refresh gets broken on HistoryFragment when device theme has changed

Set release status flags based on info from the regressing bug 1842249

:matt-tighe, since you are the author of the regressor, bug 1842249, could you take a look?

For more information, please visit BugBot documentation.

Flags: needinfo?(mtighe)
Severity: S3 → S2
Priority: -- → P2

Set release status flags based on info from the regressing bug 1842249

Assignee: nobody → giorga
Flags: needinfo?(mtighe)
Pushed by giorga@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/fd05ec6ca7e9 Refresh and delete history should work after the device theme has changed. r=android-reviewers,npoon
Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → 135 Branch

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

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

For more information, please visit BugBot documentation.

Flags: needinfo?(giorga)
Flags: needinfo?(giorga)

Verified as fixed on the latest Fenix Nightly 135.0a1 from 12/12 with the following devices:

  • Google Pixel 6 (Android 15),
  • Xiaomi Mi8 Lite (Android 10), and
  • Samsung Galaxy Tab S9 Ultra (Android 14).
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: