Closed
Bug 1227321
Opened 8 years ago
Closed 8 years ago
Settings indentation on tablets is funky
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox48 fixed)
RESOLVED
FIXED
Firefox 48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: mcomella, Assigned: litunone, Mentored)
References
Details
(Whiteboard: [lang=java][good next bug])
Attachments
(3 files)
See the attached screenshots. I imagine the items should be aligned.
Reporter | ||
Comment 1•8 years ago
|
||
Anthony, what do you think? Should these items just fall into the standard alignment of the other items?
Flags: needinfo?(alam)
Reporter | ||
Updated•8 years ago
|
Whiteboard: [lang=java][good next bug]
Comment 2•8 years ago
|
||
(In reply to Michael Comella (:mcomella) from comment #0) > Created attachment 8691066 [details] > general -> home > > See the attached screenshots. I imagine the items should be aligned. I'm not sure why "General | Home" is here... But yes, the Panel names should be aligned. E.g. "Top Sites, Bookmarks, History, etc". (In reply to Michael Comella (:mcomella) from comment #1) > Created attachment 8691068 [details] > search > > Anthony, what do you think? Should these items just fall into the standard > alignment of the other items? Yep! As above, everything should probably left-align with the "Search" text that's on the top.
Flags: needinfo?(alam)
Reporter | ||
Comment 3•8 years ago
|
||
Take a look into preferences_home & PanelsPreferenceCategory – we write a custom preference that appears to add it's own layouts which are probably missing the appropriate padding.
Use title padding. r=mcomella Review commit: https://reviewboard.mozilla.org/r/43271/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/43271/
Comment on attachment 8736406 [details] MozReview Request: Bug 1227321 - Padding fix, r=mcomella Review request updated; see interdiff: https://reviewboard.mozilla.org/r/43271/diff/1-2/
Attachment #8736406 -
Attachment description: MozReview Request: Bug 1227321 - → MozReview Request: Bug 1227321 - Padding fix, r=mcomella
Attachment #8736406 -
Flags: review?(michael.l.comella)
Reporter | ||
Updated•8 years ago
|
Attachment #8736406 -
Flags: review?(michael.l.comella) → review+
Reporter | ||
Comment 7•8 years ago
|
||
Comment on attachment 8736406 [details] MozReview Request: Bug 1227321 - Padding fix, r=mcomella https://reviewboard.mozilla.org/r/43271/#review39907 Nice! This is a really elegant solution! One comment: we'll make the correction on our side this time but in the future, can you make your patch comment more descriptive? For example, for this bug, "Corrected padding on "Panels" item in preferences." I made a push to our try test servers on the review page. Once the push goes green, you can add the "checkin-needed" keyword [1] to get your patch checked in. Note that all patches added via checkin-needed keyword need an associated green try run. Let me know if you need help reading the results. [1]: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/How_to_Submit_a_Patch#Getting_the_patch_checked_into_the_tree
Reporter | ||
Updated•8 years ago
|
Assignee: nobody → litunone
Keywords: checkin-needed
Comment 9•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/5ea59d4c9ea5
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
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
•