Closed Bug 1841574 Opened 1 year ago Closed 1 year ago

checkbox in bookmark menu item lost end gravity

Categories

(Fenix :: Bookmarks, defect)

Firefox 117
All
Android
defect

Tracking

(firefox115 wontfix, firefox116 wontfix, firefox117 wontfix, firefox118 verified)

RESOLVED FIXED
118 Branch
Tracking Status
firefox115 --- wontfix
firefox116 --- wontfix
firefox117 --- wontfix
firefox118 --- verified

People

(Reporter: mavduevskiy, Assigned: mavduevskiy)

References

(Regression)

Details

(Keywords: regression, Whiteboard: [fxdroid])

Attachments

(2 files)

The patch that fixed 1812825 bug failed to honor the previous gravity of the button. It was rightly constrained by the text element at the start and parent element at the end, but the element stopped "leaning" towards the end of the screen. The visibility of the change depends on the locale, for locales with longer words it isn't visible, but with shorter words or with wider screens the change becomes more apparent.

Component: General → Bookmarks

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

::mavduevskiy can this be reviewed and landed? Would it be safe to backport to 115 and 116?

Flags: needinfo?(mavduevskiy)

:dmeehan Mike is on PTO this week, and it looks like Sarah has an outstanding question on this PR, so I don't think it can be (or needs to be) picked up for 115 or 116 at the moment, but we can consider uplifting once he's back & has finished up the PR next week.

Severity: -- → S4
Flags: needinfo?(mavduevskiy)

Authored by mike a
https://github.com/mozilla-mobile/firefox-android/commit/d900cfb022858279f763b11c922542698014d550
[main] Bug 1841574 – changed menu item checkbox gravity from center horizontal to end

Status: NEW → RESOLVED
Closed: 1 year ago
Flags: qe-verify+
Resolution: --- → FIXED
Target Milestone: --- → 118 Branch

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

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

For more information, please visit BugBot documentation.

Flags: needinfo?(mavduevskiy)

Verified on the latest Fenix Nightly 118.0a1 from 8/10 woth the following devices:

  • Oppo Find X3 Lite (Android 11),
  • Sony Xperia Z5 Premium (Android 7.1.1), and
  • Google Pixel 6 (Android 14).

I've verified that the star icon and the "Add", or "Edit" options situated at the right side of the "Bookmarks" option from the thee-dot menu are "leaning" towards the end of the screen. The visibility of the change depends on the locale, for locales with longer words it isn't visible, but with shorter words or with wider screens the change becomes more apparent.

Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: