Tab count in tab counter is off-center after icon update
Categories
(Firefox for Android :: Design System and Theming, defect)
Tracking
()
| 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)
|
31.37 KB,
image/jpeg
|
Details | |
|
51 bytes,
text/x-github-pull-request
|
Details | Review | |
|
2.71 KB,
image/png
|
Details | |
|
11.36 KB,
image/png
|
Details | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
5.80 KB,
image/png
|
Details | |
|
48 bytes,
text/x-phabricator-request
|
phab-bot
:
approval-mozilla-beta+
|
Details | Review |
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
Updated•24 days ago
|
| Assignee | ||
Updated•24 days ago
|
Updated•24 days ago
|
Updated•24 days ago
|
Comment 1•23 days ago
|
||
The bug has a release status flag that shows some version of Firefox is affected, thus it will be considered confirmed.
Comment 2•23 days ago
|
||
Comment 3•16 days ago
|
||
Set release status flags based on info from the regressing bug 2062819
| Assignee | ||
Comment 4•15 days ago
|
||
Waiting on an updated asset from our icon designer which should fix this issue
Updated•11 days ago
|
Comment 5•9 days ago
|
||
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.
Comment 6•9 days ago
|
||
| Assignee | ||
Comment 7•9 days ago
•
|
||
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 | ||
Updated•9 days ago
|
Updated•6 days ago
|
| Assignee | ||
Comment 8•2 days ago
|
||
| Assignee | ||
Comment 9•2 days ago
|
||
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.
Comment 10•2 days ago
|
||
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
| Assignee | ||
Comment 11•2 days ago
|
||
Original Revision: https://phabricator.services.mozilla.com/D328907
Comment 12•2 days ago
|
||
Comment 13•2 days ago
|
||
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.
Comment 14•2 days ago
|
||
Comment 15•2 days ago
|
||
| bugherder | ||
Comment 16•2 days ago
|
||
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-firefox158towontfixif 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.
Comment 17•2 days ago
|
||
| bugherder | ||
Comment 18•1 day ago
|
||
:007 this needs an uplift request for beta as well
| Assignee | ||
Comment 20•1 day ago
|
||
Original Revision: https://phabricator.services.mozilla.com/D328907
Updated•1 day ago
|
Comment 21•1 day ago
|
||
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
Updated•1 day ago
|
Updated•1 day ago
|
Comment 22•1 day ago
|
||
| uplift | ||
Description
•