Closed Bug 963511 Opened 12 years ago Closed 8 years ago

[Email] Vertically Center line and string

Categories

(Firefox OS Graveyard :: Gaia::E-Mail, defect)

x86
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: epang, Assigned: pivanov)

Details

(Whiteboard: visual design)

Attachments

(4 files, 1 obsolete file)

Attached image before and after.png
Hi Pavel, Can you help implement this minor update? When adding a new email account the "All done" screen's line and string are not vertically center. They should be vertically centered between the header and buttons. See the attached screen shot (the right is the correct alignment). Thanks!
Attached image After patch screenshot (obsolete) —
Attachment #8365686 - Flags: feedback?(epang)
Attached image Vertically centered.png
Hey Pavel, thanks for working on this, but now it looks like the div line is centered. Can we center using the text and div line? See the attached, there should be 145px above and below. Thanks!
Flags: needinfo?(pivanov)
Attachment #8365686 - Flags: feedback?(epang)
Attached image After patch screenshot
Sure :) like this one?
Attachment #8365686 - Attachment is obsolete: true
Attachment #8369225 - Flags: feedback?(epang)
Flags: needinfo?(pivanov)
It seems like the patch is changing one arbitrary-ish margin rem value to another arbitrary-ish value; should we be using calc() or flexbox or something to more confidently center things if that's the goal? Especially if we have specific behaviour for when the string translates to something that takes up multiple lines.
Comment on attachment 8369225 [details] After patch screenshot looks good Pavel! But can you look into Andrew's comments? Thanks!
Attachment #8369225 - Flags: feedback?(epang) → feedback+
Flags: needinfo?(pivanov)
Hey Andrew, I agree with you :) we will work on the Visual Design Refresh for v1.4 and v1.5 and I'm not sure if there will be some changes of the confirm/modal dialogs and if 'yes' I will definitely do this with flex if 'no' we will need to plan some time to fix old [BB] ... but for now we don't have time to prepare this [BB] with flex or some other tricks. I think Eric can give us more light on this one.
Flags: needinfo?(pivanov) → needinfo?(epang)
(In reply to Pavel Ivanov [:ivanovpavel] from comment #7) > Hey Andrew, > > I agree with you :) > > we will work on the Visual Design Refresh for v1.4 and v1.5 and I'm not sure > if there will be some changes of the confirm/modal dialogs and if 'yes' I > will definitely do this with flex if 'no' we will need to plan some time to > fix old [BB] ... but for now we don't have time to prepare this [BB] with > flex or some other tricks. I think Eric can give us more light on this one. Hey Guys, I believe there will be changes to the confirm/modal dialogs but I don't think they will be for v.1.4.
Flags: needinfo?(epang)
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: