Closed
Bug 1430122
Opened 6 years ago
Closed 6 years ago
Custom tab close button is too large compared to the other buttons
Categories
(Firefox for Android Graveyard :: Custom Tabs, defect, P5)
Tracking
(fennec+, firefox59 wontfix, firefox60 wontfix, firefox61 wontfix, firefox62 verified, firefox63 verified)
VERIFIED
FIXED
Firefox 63
People
(Reporter: ohorvath, Assigned: andrei.a.lazar)
References
Details
(Keywords: good-first-bug, Whiteboard: --do_not_change--[priority:high])
Attachments
(3 files, 1 obsolete file)
Device: Samsung Galaxy S8 (Android 7.0) Build: Nightly 59.0a1 (2018-01-12); Steps to reproduce: 1. Open a link in a custom tab (default browser: Nightly). 2. Check the close button. Expected result: See the "previous button" screenshot. Actual result: The new close button is too large compared to the previous and the other buttons used.
Reporter | ||
Comment 1•6 years ago
|
||
Comment 2•6 years ago
|
||
Please help advise. Thanks!
Updated•6 years ago
|
tracking-fennec: ? → +
[triage] Non-critical.
Priority: P2 → P3
Comment 5•6 years ago
|
||
I think it's using the wrong resolution images. Could someone help provide the name of the icon asset (image)? so i could check, thanks!
Flags: needinfo?(chuang)
Updated•6 years ago
|
status-firefox60:
--- → affected
status-firefox61:
--- → affected
It seems this was introduced with FF59, it was correct in FF58, at least.
NI Barbara and Andreas, should we fix this for 61 or 62?
Flags: needinfo?(bbermes)
Flags: needinfo?(abovens)
Comment 9•6 years ago
|
||
I would like to get this fixed soon, as it's very visible. Can it be prioritized?
Flags: needinfo?(sdaswani)
Comment 10•6 years ago
|
||
Andreas, the Softvision team is currently prioritizing the Android O work: https://trello.com/c/izAPXSyJ/506-fennec-target-sdk-version-62 I can have one of them work on this issue over that work if you feel it is a bigger priority.
Flags: needinfo?(sdaswani) → needinfo?(abovens)
Comment 12•6 years ago
|
||
Vlad, can you please get this on your prioritized list (under the O work)?
Flags: needinfo?(bbermes) → needinfo?(vlad.baicu)
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → andrei.a.lazar
Updated•6 years ago
|
Flags: needinfo?(vlad.baicu)
Comment 13•6 years ago
|
||
Andrei, can you prioritize this immediately after all the API upgrade crash work?
Flags: needinfo?(andrei.a.lazar)
Whiteboard: [priority:low] → --do_not_change--[priority:high]
Assignee | ||
Comment 14•6 years ago
|
||
Sure thing, will fix this ASAP!
Comment 15•6 years ago
|
||
Thanks, Andrei.
Comment 16•6 years ago
|
||
Re-triaging per https://bugzilla.mozilla.org/show_bug.cgi?id=1473195 Needinfo :susheel if you think this bug should be re-triaged.
Priority: P3 → P5
Assignee | ||
Comment 17•6 years ago
|
||
Comment 18•6 years ago
|
||
Comment on attachment 9003173 [details] Bug 1430122 Custom tab close button is too large compared to the other buttons r?sdaswani Jim Chen [:jchen] [:darchons] has approved the revision.
Attachment #9003173 -
Flags: review+
Assignee | ||
Updated•6 years ago
|
Flags: needinfo?(andrei.a.lazar)
Keywords: checkin-needed
Comment 19•6 years ago
|
||
Please review your patch for this bug. Phabricator says the following: This revision is not associated with a repository. In order to land, a revision must be associated with a repository on Phabricator. This diff does not have the proper author information uploaded to Phabricator. This can happen if the diff was created using the web UI, or a non standard client. The author should re-upload the diff to Phabricator using the "arc diff" command.
Flags: needinfo?(andrei.a.lazar)
Keywords: checkin-needed
Assignee | ||
Comment 20•6 years ago
|
||
Resized the close button manually.
Comment 21•6 years ago
|
||
Comment on attachment 9004239 [details] Bug 1430122 Custom tab close button is too large compared to the other buttons. Vlad Baicu has approved the revision.
Attachment #9004239 -
Flags: review+
Assignee | ||
Updated•6 years ago
|
Flags: needinfo?(andrei.a.lazar)
Keywords: checkin-needed
Comment 22•6 years ago
|
||
Pushed by dluca@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c8df42395ed8 Custom tab close button is too large compared to the other buttons. r=VladBaicu
Keywords: checkin-needed
Comment 23•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c8df42395ed8
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Updated•6 years ago
|
status-firefox62:
--- → fix-optional
Flags: qe-verify+
Updated•6 years ago
|
Attachment #9003173 -
Attachment is obsolete: true
Reporter | ||
Comment 24•6 years ago
|
||
Verified as fixed on Nightly 63 (2018-29-08). Devices: Sony Xperia Z5 Premium (Android 6.0.1) Google Pixel (Android 9)
Flags: qe-verify+
Comment 25•6 years ago
|
||
Seems like a trivial ride-along fix we could take for RC2. Please nominate this for release approval if you agree.
Flags: needinfo?(andrei.a.lazar)
Assignee | ||
Comment 26•6 years ago
|
||
Comment on attachment 9004239 [details] Bug 1430122 Custom tab close button is too large compared to the other buttons. Approval Request Comment [Feature/Bug causing the regression]: Custom tab close button is too large [User impact if declined]: This causes a bad user experience [Is this code covered by automated tests?]: No [Has the fix been verified in Nightly?]: Yes [Needs manual test from QE? If yes, steps to reproduce]: No [List of other uplifts needed for the feature/fix]: No [Is the change risky?]: No [Why is the change risky/not risky?]: Because it's just an image [String changes made/needed]: No
Flags: needinfo?(andrei.a.lazar) → needinfo?(ryanvm)
Attachment #9004239 -
Flags: approval-mozilla-release?
Comment 27•6 years ago
|
||
Comment on attachment 9004239 [details] Bug 1430122 Custom tab close button is too large compared to the other buttons. Simple polish fix for custom tabs. Approved for 62 RC2.
Flags: needinfo?(ryanvm)
Attachment #9004239 -
Flags: approval-mozilla-release? → approval-mozilla-release+
Updated•6 years ago
|
Flags: qe-verify+
Comment 28•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-release/rev/7682ad2849f6
Reporter | ||
Comment 29•6 years ago
|
||
Verified as fixed on RC 62.0 build 2. Device: Sony Xperia Z5 Premium (Android 6.0.1) Google Pixel (Android 9)
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•