Closed
Bug 916341
Opened 12 years ago
Closed 11 years ago
[Notes] When attaching a media to a note and email the note it does not attach the picture
Categories
(Firefox OS Graveyard :: Gaia::Notes, defect, P2)
Firefox OS Graveyard
Gaia::Notes
Tracking
(Not tracked)
RESOLVED
FIXED
1.3 C2/1.4 S2(17jan)
People
(Reporter: jaime.dolado, Assigned: yor)
References
Details
Attachments
(2 files)
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:23.0) Gecko/20100101 Firefox/23.0 (Beta/Release)
Build ID: 20130814063812
Steps to reproduce:
Open the Notes App
Attach a picture and a text
and email the note
Actual results:
It does not attach the picture in the email.
Expected results:
Send the picture within the email
Updated•12 years ago
|
Component: General → Gaia::E-Mail
| Reporter | ||
Updated•12 years ago
|
Priority: -- → P2
Comment 1•12 years ago
|
||
Attachments work fine in Notes, but do not work in Notes+.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(matias)
Updated•12 years ago
|
Assignee: nobody → dpreston
Status: NEW → ASSIGNED
Comment 2•12 years ago
|
||
Problem persists. I reproduced this on a Keon geeksphone device with nightly build 20131028225522 FFOS v1.2
Comment 3•12 years ago
|
||
Reproduced bug on:
-Keon FFOS v1.1 and v1.2
-Huawei FFOS v1.1
Ran and Matias, could you please provide an estimated fix date for this bug?
Assignee: dpreston → matias
Updated•11 years ago
|
Component: Gaia::E-Mail → Gaia::Notes
Updated•11 years ago
|
Assignee: matias → yor
Comment 5•11 years ago
|
||
according to Yan this is a duplicate, so I'm clearing the needinfo.
Flags: needinfo?(matias)
Attachment #8359627 -
Flags: review?(matias)
Comment 7•11 years ago
|
||
After applying Yor's patch im getting the entire note as an .html attachment with the image as inline.
is this the desired result?
Flags: needinfo?(yor)
Comment 8•11 years ago
|
||
The Mail app currently does not support HTML in the body. The decision was made to have a stripped (text only) version of the note in the body and include the full HTML as an attachment.
Flags: needinfo?(yor)
Comment 10•11 years ago
|
||
Comment on attachment 8359627 [details] [review]
Pull request
r+ for Matias
Attachment #8359627 -
Flags: review?(matias) → review+
Keywords: checkin-needed
Comment 11•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 1.3 C2/1.4 S2(17jan)
You need to log in
before you can comment on or make changes to this bug.
Description
•