Closed
Bug 1016541
Opened 10 years ago
Closed 10 years ago
Thread and Composer. The input placeholder text (Message) should be italic
Categories
(Firefox OS Graveyard :: Gaia::SMS, defect)
Tracking
(b2g-v2.0 fixed)
RESOLVED
FIXED
2.0 S3 (6june)
Tracking | Status | |
---|---|---|
b2g-v2.0 | --- | fixed |
People
(Reporter: vicky, Assigned: paco)
References
Details
Attachments
(2 files)
46 bytes,
text/x-github-pull-request
|
steveck
:
review+
|
Details | Review |
22.03 KB,
image/png
|
vicky
:
ui-review+
|
Details |
The input placeholder text (Message) should be italic, the writing text though, should be regular style. It should have a 1.5 rem gap from the border of the screen And ideally the text should be a bit more engaging like "write your message here..."
Updated•10 years ago
|
blocking-b2g: --- → 2.0?
Comment 1•10 years ago
|
||
triage: 2.0+ as the UI should align with VR design
blocking-b2g: 2.0? → 2.0+
Comment 2•10 years ago
|
||
(In reply to Victoria Gerchinhoren [:vicky] from comment #0) > The input placeholder text (Message) should be italic, the writing text > though, should be regular style. > It should have a 1.5 rem gap from the border of the screen > And ideally the text should be a bit more engaging like "write your message > here..." wesley : all these seem's nice to have's but why would this be a blocker ? If this is not done by FL and uplift would be a right way to go, but this does not sound like a release blocker to me as written.
blocking-b2g: 2.0+ → 2.0?
Comment 3•10 years ago
|
||
This is the expected design. https://bug950175.bugzilla.mozilla.org/attachment.cgi?id=8394784
Comment 4•10 years ago
|
||
Bhavana, I expect that we can request approval between FL and FC, right? So we don't need this to be a blocker, but we should just request an approval when ready, is that right?
Reporter | ||
Comment 5•10 years ago
|
||
Hi, The margin on the left hand side of the input field looks OK now, but the placeholder's font still needs to be "italic" style to match the rest of input fields in the OS. This means a very small change, right?
Comment 6•10 years ago
|
||
Changing the title according to Vicky's last comment.
Summary: Thread and Composer. Fix the input field margins. → Thread and Composer. The input placeholder text (Message) should be italic
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → pacorampas
Assignee | ||
Comment 7•10 years ago
|
||
Assignee | ||
Comment 8•10 years ago
|
||
Attachment #8434899 -
Flags: ui-review?(vpg)
Reporter | ||
Updated•10 years ago
|
Attachment #8434899 -
Flags: ui-review?(vpg) → ui-review+
Assignee | ||
Updated•10 years ago
|
Attachment #8434898 -
Flags: review?(schung)
Comment 9•10 years ago
|
||
Thanks! r=me in master: bd86b8d68389df7656fc426ea59995d7dfbfa1cb
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Attachment #8434898 -
Flags: review?(schung) → review+
Updated•10 years ago
|
blocking-b2g: 2.0? → ---
Updated•10 years ago
|
status-b2g-v2.0:
--- → fixed
Target Milestone: --- → 2.0 S3 (6june)
You need to log in
before you can comment on or make changes to this bug.
Description
•