Status
People
(Reporter: hub, Assigned: hub)
Tracking
Firefox Tracking Flags
(Not tracked)
Details
Attachments
(1 attachment)
|
46 bytes,
text/x-github-pull-request
|
pdahiya
:
review+
|
Details | Review | Splinter Review |
Follow-up to bug 1178812 : we need to copy Exif for the share action. See frames.js:shareSingleItem()
|
(Assignee) | |
Updated•3 years ago
|
Assignee: nobody → hub
|
||
Comment 1•3 years ago
|
||
Created attachment 8705807 [details] [review] [gaia] hfiguiere:bug1237367-exif-copy-share > mozilla-b2g:master
|
|
(Assignee) | |
Updated•3 years ago
|
Attachment #8705807 -
Flags: review?(dflanagan)
|
|
(Assignee) | |
Comment 2•3 years ago
|
||
Comment on attachment 8705807 [details] [review] [gaia] hfiguiere:bug1237367-exif-copy-share > mozilla-b2g:master Punam, if you don't mind reviewing this. Thanks !
Attachment #8705807 -
Flags: review?(dflanagan) → review?(pdahiya)
|
||
Comment 3•3 years ago
|
||
Hi Hub I am following below steps to test exif copy on single file share action. 1. Share file in gallery with messaging app 2. Send the message with the shared file. 3. Open the shared message and click on the attachment and save the attached image. 4. Saved image should show up in gallery With the patch, I was expecting to see EXIF info with the saved image but I don't see it. Am I missing something? I am using https://github.com/recurser/exif-orientation-examples/blob/master/Portrait_6.jpg to test the above flow. Thanks!
Flags: needinfo?(hub)
|
|
||
Comment 4•3 years ago
|
||
Comment on attachment 8705807 [details] [review] [gaia] hfiguiere:bug1237367-exif-copy-share > mozilla-b2g:master Issue reported in comment #3 resolved for me with re-flash of flame. EXIF data is seen on shared image. Thanks!
Attachment #8705807 -
Flags: review?(pdahiya) → review+
|
|
(Assignee) | |
Comment 5•3 years ago
|
||
Merged https://github.com/mozilla-b2g/gaia/commit/aee2763847b1b289f4876b04de78f06ee31e22c5
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Flags: needinfo?(hub)
Resolution: --- → FIXED
|
|
(Assignee) | |
Updated•3 years ago
|
Target Milestone: --- → 2.6 S7 - 2/12
You need to log in
before you can comment on or make changes to this bug.
Description
•