Closed Bug 2068594 Opened 24 days ago Closed 2 days ago

Tab count in tab counter is off-center after icon update

Categories

(Firefox for Android :: Design System and Theming, defect)

Firefox 157
All
Android
defect

Tracking

()

RESOLVED FIXED
159 Branch
Tracking Status
firefox155 --- unaffected
firefox156 --- wontfix
firefox157 --- fix-optional
firefox158 --- fixed
firefox159 --- fixed

People

(Reporter: 1zgoo2hl2, Assigned: 007)

References

(Regression)

Details

(Keywords: regression, Whiteboard: [fxdroid][android-core])

Attachments

(8 files)

Attached image IMG_20260901_131021.jpg —

User Agent: Mozilla/5.0 (Android 16; Mobile; rv:140.0) Gecko/140.0 Firefox/140.0

Steps to reproduce:

The number indicating the number of open tabs is off-centre

Keywords: regression
Regressed by: 2062819
Severity: -- → S3
Summary: Decentralised tab indicator → Tab count in tab counter is off-center after icon update
Whiteboard: [fxdroid][android-core]

The bug has a release status flag that shows some version of Firefox is affected, thus it will be considered confirmed.

Status: UNCONFIRMED → NEW
Ever confirmed: true

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

Waiting on an updated asset from our icon designer which should fix this issue

Some feedback, please ensure this is passed to the decision-maker for icon design.

The new "squished rectangle" with bowed edges is unattractive, even if aligned properly. The former icon square was roughly 1:1.5 ratio while the new icon is 1:2. I am not using a landscape-orientation device like the horizontal icon implies.

Please consider additional vertical height and more rigid edges for the tab count outline, similar to the former icon shape.

Attached image image.png —

The updated assets were sent over from UX this morning. To fix this visual bug, I'll proceed with these.

https://github.com/FirefoxUX/acorn-icons/blob/main/icons/mobile/24/svg/tab-24.svg
https://github.com/FirefoxUX/acorn-icons/blob/main/icons/mobile/24/svg/infinite-tabs-24.svg

Edit: I found out these are still the old assets. I'm waiting on some correct ones from UX

Assignee: nobody → nbond
Status: NEW → ASSIGNED

Github was not landing the fix and I mistook that for Lando being down, when there's a bug with Github approvals. I've made a new patch on phabricator and it's in the landing queue.

firefox-release Uplift Approval Request

  • User impact if declined/Reason for urgency: This is a highly visible icon in Firefox and is resulting in some loud user feedback.
  • Code covered by automated testing?: yes
  • Fix verified in Nightly?: no
  • Needs manual QE testing?: yes
  • Steps to reproduce for manual QE testing: The tab counter icon has been updated for the toolbar and tabs tray for both the 0-99 tabs scenario and 100+/infinity scenario.
  • Risk associated with taking this patch: low
  • Explanation of risk level: This patch updates the icon XML of a vector asset
  • String changes made/needed?: n/a
  • Is Android affected?: yes
Attachment #9648071 - Flags: approval-mozilla-release?
Flags: qe-verify+
Pushed by nbond@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/b6513b364538 https://hg.mozilla.org/integration/autoland/rev/b9ab0932ebe8 Update the tab counter icons to the latest design r=android-reviewers,jdelorenzo

I've built and tested D328907/D328909. The alignment is fixed, thank you.

I still maintain that the new icon's "squished" shape is an overall downgrade, it visually communicates that the user is viewing on a TV/CRT device, not the case for a mobile device. For posterity, I've opened BUG 2073311 to discuss this design regression caused by the new shape.

Status: ASSIGNED → RESOLVED
Closed: 2 days ago
Resolution: --- → FIXED
Target Milestone: --- → 159 Branch

The patch landed in nightly and beta is affected.

:007, please make an uplift decision for beta:

  • Nominate the patch for beta uplift approval if the fix should be included in this release, or
  • Set status-firefox158 to wontfix if the fix can wait for the next release.

See Requesting an Uplift for documentation on how to request an uplift.

For more information, please visit BugBot documentation.

Flags: needinfo?(nbond)

:007 this needs an uplift request for beta as well

Added! Thanks, Dianna

Flags: needinfo?(nbond)
Attachment #9648530 - Flags: approval-mozilla-beta?

firefox-beta Uplift Approval Request

  • User impact if declined/Reason for urgency: This is a highly visible icon in Firefox and is resulting in some loud user feedback.
  • Code covered by automated testing?: no
  • Fix verified in Nightly?: yes
  • Needs manual QE testing?: yes
  • Steps to reproduce for manual QE testing: The tab counter icon has been updated for the toolbar and tabs tray for both the 0-99 tabs scenario and 100+/infinity scenario.
  • Risk associated with taking this patch: low
  • Explanation of risk level: This patch updates the icon XML of a vector asset
  • String changes made/needed?: n/a
  • Is Android affected?: yes
Attachment #9648530 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: