Closed
Bug 847849
Opened 11 years ago
Closed 11 years ago
Hostname in site settings dialog is misaligned
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox21 verified, firefox22 verified)
VERIFIED
FIXED
Firefox 22
People
(Reporter: Margaret, Assigned: Margaret)
References
Details
Attachments
(2 files, 1 obsolete file)
57.87 KB,
image/png
|
Details | |
1.00 KB,
patch
|
wesj
:
review+
bajaj
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
See screenshot. Also, the ellipsis in the title looks awkward, but this would be fixed if we fix bug 786999.
Comment 2•11 years ago
|
||
I'm not sure how removing the ellipsis fixes this? Its likely fallout from bug 844834.
Blocks: 844834
Assignee | ||
Comment 3•11 years ago
|
||
I guess I wasn't clear. Removing the ellipsis just fixes the fact that the ellipsis in the title is awkward :) We still need to do something to fix the host name alignment.
Assignee | ||
Comment 5•11 years ago
|
||
The patch in bug 844834 got rid of this padding. I also found that the title and subtitle are both black on black on gingerbread. That also needs a fix.
Assignee: nobody → margaret.leibovic
Attachment #723702 -
Flags: review?(wjohnston)
Comment 6•11 years ago
|
||
Comment on attachment 723702 [details] [diff] [review] put back padding that was removed The patch at: https://hg.mozilla.org/mozilla-central/rev/5668dcdcdbdd has padding on the left and right. I moved them to styles and then supposedly moved them back (which apparently means I didn't move them back). Might as well put both back.
Assignee | ||
Comment 7•11 years ago
|
||
Attachment #723702 -
Attachment is obsolete: true
Attachment #723702 -
Flags: review?(wjohnston)
Attachment #723705 -
Flags: review?(wjohnston)
Comment 8•11 years ago
|
||
Comment on attachment 723705 [details] [diff] [review] put back padding on both sides Review of attachment 723705 [details] [diff] [review]: ----------------------------------------------------------------- I realize now what happened. I moved these to styles (where I still think they should live) then I moved them back, but since I was using inheritence in the styles so we didn't have to duplicate this all over, I forgot to add them back here (aka if they were styles this would have been prettier and easier).
Attachment #723705 -
Flags: review?(wjohnston) → review+
Assignee | ||
Comment 9•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/905682481543
Assignee | ||
Comment 10•11 years ago
|
||
Comment on attachment 723705 [details] [diff] [review] put back padding on both sides [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 844834 User impact if declined: site settings dialog title looks bad Testing completed (on m-c, etc.): just landed on inbound Risk to taking this patch (and alternatives if risky): low-risk, small style change String or UUID changes made by this patch: n/a
Attachment #723705 -
Flags: approval-mozilla-aurora?
Comment 11•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/905682481543
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 22
Comment 12•11 years ago
|
||
Comment on attachment 723705 [details] [diff] [review] put back padding on both sides Approving the low risk uplift of a minor alignment glitch in site settings dialog.
Attachment #723705 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 13•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/6f2799b58776
status-firefox21:
--- → fixed
status-firefox22:
--- → fixed
Comment 14•11 years ago
|
||
Verified on Firefox for Android 21.0b7 and Aurora 22.0a2 (2013-05-07) using: Nexus 4 (4.2.2)
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
•