Closed Bug 1043527 Opened 7 years ago Closed 7 years ago

email UI: "Show external images" styling

Categories

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

x86
macOS
defect
Not set
normal

Tracking

(feature-b2g:2.1)

VERIFIED FIXED
2.1 S3 (29aug)
feature-b2g 2.1

People

(Reporter: doliver, Assigned: jrburke)

References

Details

(Keywords: feature, Whiteboard: [tako][2.1-feature-qa+])

Attachments

(4 files, 2 obsolete files)

Update the styling of the button to load external images. See design spec in bug 1028224.
Summary: email UI: Read message with external images → email UI: "Show external images" styling
Whiteboard: [tako]
Attached image external-images.png (obsolete) —
Attached is a screenshot is an experiment I did to meet the goals of this ticket. Since this ticket is fairly new, there is not visual design spec for this, so I could have gotten it horribly wrong.

So :peko, in the case where you did not want this sort of styling, feel free to discard this, and in that case, this ni request is for the visual spec that you desire.

The image on the left is what is shown when the images will be fetched from a remote web site. The second image shows how it will look if there are other attachments in the email, and if the images for the email message are included in the email, but just not shown.

I used the View image used for attachments here, but again, this is very speculative, just more to start the conversation.
Flags: needinfo?(pchen)
Hi James,

Thanks for your work~ actually your screenshot is quite the same as my design. Just need to modify some details. I still waiting for the feedback from Peter and Patryk, and will update spec asap.
thanks
Flags: needinfo?(pchen)
QA Whiteboard: [2.1-feature-qa+]
QA Whiteboard: [2.1-feature-qa+] → [COM=Productivity]
QA Whiteboard: [COM=Productivity] → [COM=Gaia::E-Mail]
QA Whiteboard: [COM=Gaia::E-Mail] → [COM=Gaia::E-Mail][2.1-feature-qa+]
Flags: in-moztrap?(edchen)
QA Contact: edchen
QA Whiteboard: [COM=Gaia::E-Mail][2.1-feature-qa+] → [COM=Gaia::E-Mail]
Whiteboard: [tako] → [tako][2.1-feature-qa+]
No testcase, I will verify UI directly.
Flags: in-moztrap?(edchen) → in-moztrap-
Target Milestone: --- → 2.1 S3 (29aug)
Attached file spec_external_image.zip (obsolete) —
Hi James,

Zip file including spec and icon image.
Please see attached zip file for your reference.
thanks!
updated more detail
Attachment #8472871 - Attachment is obsolete: true
Attached image external-images.png
Attached is what I have implemented now in a branch, based on the spec. It shows the two types of messages that could show up there. Notes:

1) For the bottom one pixel border, I used #f2f2f1 instead of #f5f5f4, because the existing border under the subject line is use #f2f2f1.

2) It is not exactly 55px height, more like 52px, based on font sizing results and using the space that comes naturally. I did not want to set a hard height because the text needs to be able to expand even potentially to three lines based on the localizations.

3) The zip file with the spec only included icon_show_external_image.png. I will need the @1.5, 2 and 2.25x versions of the icon too.
Attachment #8463618 - Attachment is obsolete: true
Attachment #8473394 - Flags: ui-review?(pchen)
Attachment #8473394 - Flags: ui-review?(jhuang)
Comment on attachment 8473394 [details]
external-images.png

Looks good to me! Thanks James!
Attachment #8473394 - Flags: ui-review?(jhuang) → ui-review+
Comment on attachment 8473394 [details]
external-images.png

Looks good! Thank you James!
Attachment #8473394 - Flags: ui-review?(pchen) → ui-review+
Attached file icon.zip
uploaded other resolution.
Attached file GitHub pull request
Pull request, asuth already r+ in the pull request, but will formalize that with the flags here.
Attachment #8473891 - Flags: review?(bugmail)
Assignee: nobody → jrburke
Comment on attachment 8473891 [details] [review]
GitHub pull request

It's traditionally allowed to directly set the review flag to + if you indicate in the comment who did it and via what communication mechanism.  like "r=asuth from the pull request" or "r=asuth on IRC".
Attachment #8473891 - Flags: review?(bugmail) → review+
Merged in master:
https://github.com/mozilla-b2g/gaia/commit/5e37ce8c7f0bf6992911d77eb31513db37656aba

from pull request:
https://github.com/mozilla-b2g/gaia/pull/22937
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
[Environment]
Gaia      e424c85eda87a40c0fa64d6a779c3fa368bf770b
Gecko     https://hg.mozilla.org/mozilla-central/rev/daa84204a11a
BuildID   20140824160205
Version   34.0a1
ro.build.version.incremental=94
ro.build.date=Tue May 20 09:29:20 CST 2014

[Result]
PASS
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.