Last Comment Bug 845067 - Send Email screen update
: Send Email screen update
Status: RESOLVED INVALID
visual design, UX-P1, yedo, landed in...
: polish
Product: Firefox OS
Classification: Client Software
Component: Gaia::E-Mail (show other bugs)
: unspecified
: All Other
: -- normal (vote)
: ---
Assigned To: Pavel Ivanov [:ivanovpavel][:pivanov] UX
: Naoki Hirata :nhirata (please use needinfo instead of cc)
Mentors:
Depends on:
Blocks: 841512
  Show dependency treegraph
 
Reported: 2013-02-25 14:51 PST by Eric Pang [:epang] UX
Modified: 2013-04-08 08:15 PDT (History)
6 users (show)
epang: needinfo?
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
+
affected


Attachments
Sending email screen mock up (104.88 KB, image/png)
2013-02-25 14:51 PST, Eric Pang [:epang] UX
no flags Details
Updated Sending email screen mock up (196.95 KB, image/png)
2013-02-26 15:56 PST, Eric Pang [:epang] UX
no flags Details
After patch (212.40 KB, image/png)
2013-03-10 03:46 PDT, Pavel Ivanov [:ivanovpavel][:pivanov] UX
no flags Details
patch for Gaia repo (181 bytes, text/html)
2013-03-10 03:51 PDT, Pavel Ivanov [:ivanovpavel][:pivanov] UX
sjochimek: review+
Details

Description Eric Pang [:epang] UX 2013-02-25 14:51:23 PST
Created attachment 718099 [details]
Sending email screen mock up

Changes:
1.  Consistent system font treatment
3.  Spinner moved to right of text (to preserve left alignment of type)
Comment 1 Eric Pang [:epang] UX 2013-02-26 15:56:31 PST
Created attachment 718739 [details]
Updated Sending email screen mock up

See attached image for list of changes
Comment 2 Pavel Ivanov [:ivanovpavel][:pivanov] UX 2013-03-10 03:46:29 PDT
Created attachment 723186 [details]
After patch
Comment 3 Pavel Ivanov [:ivanovpavel][:pivanov] UX 2013-03-10 03:51:14 PDT
Created attachment 723187 [details]
patch for Gaia repo
Comment 4 Eric Pang [:epang] UX 2013-03-11 08:19:14 PDT
(In reply to Pavel Ivanov [:ivanovpavel] from comment #2)
> Created attachment 723186 [details]
> After patch

Looks good! thx!
Comment 5 Sam Joch [:samjoch] 2013-03-13 07:12:49 PDT
Comment on attachment 723187 [details]
patch for Gaia repo

ok according to BB it's a r+ for me.
Comment 6 Eric Pang [:epang] UX 2013-03-20 07:49:43 PDT
LG request for font updates.
Comment 7 Lukas Blakk [:lsblakk] use ?needinfo 2013-03-21 09:27:57 PDT
Mass edit to set tracking-b2g18+ for these UX bugs that were called out for v1.1
Comment 8 Naoki Hirata :nhirata (please use needinfo instead of cc) 2013-04-04 16:49:30 PDT
Not sure why the E is capitalized at the end? 

https://github.com/pivanov/gaia/commit/7e5503db6705894d9c6decc1a602b7f719193c03#L0R1078


Master build : 2013-04-04-03-09-55
name="mozilla-central" revision="c232bec6974d"
name="integration/gaia-central" revision="d50316e8e9b9"
name="gecko.git" revision="4d9b24f0cb7cabfa1ceb4d2953e993e6af3ca1c4"
name="gaia.git" revision="29660ec0a4ad7c90d22e33852e1601d45f781e03"
Comment 9 Andrew Sutherland [:asuth] 2013-04-04 16:59:32 PDT
(In reply to Naoki Hirata :nhirata (please use needinfo instead of cc) from comment #8)
> Not sure why the E is capitalized at the end? 

That's not the localized string.  It's a string in the HTML to provide context but make it clear it's not the localized string and that if you're seeing it, we have a localization problem.
Comment 10 Eric Pang [:epang] UX 2013-04-05 11:45:05 PDT
(In reply to Naoki Hirata :nhirata (please use needinfo instead of cc) from comment #8)
> Not sure why the E is capitalized at the end? 
> 
> https://github.com/pivanov/gaia/commit/
> 7e5503db6705894d9c6decc1a602b7f719193c03#L0R1078
> 
> 
> Master build : 2013-04-04-03-09-55
> name="mozilla-central" revision="c232bec6974d"
> name="integration/gaia-central" revision="d50316e8e9b9"
> name="gecko.git" revision="4d9b24f0cb7cabfa1ceb4d2953e993e6af3ca1c4"
> name="gaia.git" revision="29660ec0a4ad7c90d22e33852e1601d45f781e03"

Hi Naoki, the capital E isn't seen anywhere and as Andrew said, it's used to provide context.
Comment 11 Victoria Gerchinhoren [:vicky] 2013-04-08 01:56:48 PDT
Hi Eric,
This update was not necesary, can you please explain why are you requesting an updated design? AFAK this is following the specs and using the Building Blocks provided for activity indicators. 
See: http://buildingfirefoxos.com/building-blocks/progress-and-activity/

If you can clarify the need for this change it would be great.
Comment 12 Eric Pang [:epang] UX 2013-04-08 08:15:03 PDT
Hi Victoria, I discussed this with Przemek and we agree that this can stay as is.  The reason we updated the design was to try and find a consistent way to display system msgs.  I'll email you with more details soon. Thanks!

Note You need to log in before you can comment on or make changes to this bug.