Status
()
People
(Reporter: liuche, Assigned: lucasr)
Tracking
Firefox Tracking Flags
(Not tracked)
Details
Attachments
(3 attachments)
Created attachment 8476870 [details] Mock-up: Homepage design Splitting this off from bug 1021751, because there seems to be an Android bug that affects the padding of the title strip such that text in the titlebar doesn't center vertically correctly.
(Assignee) | ||
Comment 1•4 years ago
|
||
Found it, PagerTabStrip sets a minimum padding bottom in order to ensure none of the children will be placed under the indicator. See: https://github.com/android/platform_frameworks_support/blob/master/v4/java/android/support/v4/view/PagerTabStrip.java#L89 https://github.com/android/platform_frameworks_support/blob/master/v4/java/android/support/v4/view/PagerTabStrip.java#L151 The cleanest workaround here is to override getPaddingBottom() to return 0. Patch coming in a minute.
(Assignee) | ||
Comment 2•4 years ago
|
||
Created attachment 8477314 [details] [diff] [review] Increate height of HomePagerTabStrip to 40dp (r=liuche)
(Assignee) | ||
Comment 3•4 years ago
|
||
Comment on attachment 8477314 [details] [diff] [review] Increate height of HomePagerTabStrip to 40dp (r=liuche) I'm assuming the new height is 40dp here. Let me know if it's something else.
Attachment #8477314 -
Flags: review?(liuche)
(Assignee) | ||
Comment 4•4 years ago
|
||
Created attachment 8477318 [details] Screenshot for the record
(Reporter) | ||
Comment 5•4 years ago
|
||
Comment on attachment 8477314 [details] [diff] [review] Increate height of HomePagerTabStrip to 40dp (r=liuche) Review of attachment 8477314 [details] [diff] [review]: ----------------------------------------------------------------- Great! This is actually the exact patch I had in my queue too :)
Attachment #8477314 -
Flags: review?(liuche) → review+
(Reporter) | ||
Comment 6•4 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/cb24d9bb372e Landed this patch because I want it to be with bug 1021751.
Assignee: nobody → lucasr.at.mozilla
Status: NEW → ASSIGNED
Target Milestone: --- → Firefox 34
Comment 7•4 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/cb24d9bb372e
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•