Closed Bug 1018322 Opened 11 years ago Closed 10 years ago

Checkbox "Use as default ringtone" looks bad when translation spans on multiple lines

Categories

(Firefox OS Graveyard :: Gaia::Ringtones, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: flod, Assigned: squib)

References

Details

Attachments

(4 files)

Attached image Screenshot Keon
See attachment: horizontal alignment of the second line is off (minor), the line-height is too big.
Attached file Fix it
Here's a fix. (A part of me thinks I should just redo everything with flexbox, but this works and that's enough for me.)
Assignee: nobody → squibblyflabbetydoo
Status: NEW → ASSIGNED
Attachment #8433638 - Flags: ui-review?(tshakespeare)
Attachment #8433638 - Flags: review?(dflanagan)
Attached image Screenshot of the fix
Comment on attachment 8433638 [details] [review] Fix it Took a look at the screenshot and things look much better. Thanks Jim!
Attachment #8433638 - Flags: ui-review?(tshakespeare) → ui-review+
Comment on attachment 8433638 [details] [review] Fix it looks good to me
Attachment #8433638 - Flags: review?(dflanagan) → review+
Landed: https://github.com/mozilla-b2g/gaia/pull/19962 Setting in-testsuite to "-" because this is just a CSS change.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Attached file Fix alignment
Whoops, I forgot to save this part of the file originally. This just makes the text centered a little nicer. I verified that this looks the same as it did before I "fixed" this bug. :)
Attachment #8433768 - Flags: review?(dflanagan)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attachment #8433768 - Flags: review?(dflanagan) → review+
Status: REOPENED → RESOLVED
Closed: 10 years ago10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: