Closed
Bug 1177201
Opened 9 years ago
Closed 9 years ago
Saving pictures doesn't always prompt for Photos permission
Categories
(Firefox for iOS :: Browser, defect)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
fxios | + | --- |
People
(Reporter: rnewman, Assigned: st3fan)
Details
(Whiteboard: noteworthy)
Attachments
(2 files, 1 obsolete file)
A user reported that he can't save pictures, was never prompted for the appropriate permission, and doesn't see it in Settings > Firefox. We should double check that we're requesting that permission.
Updated•9 years ago
|
tracking-fxios:
--- → ?
Reporter | ||
Updated•9 years ago
|
Attachment #8629573 -
Flags: review?(sarentz)
Assignee | ||
Comment 2•9 years ago
|
||
Than you very much for this patch :allengn! I am cleaning this up so that we can land this today. Just changing the strings to be localized.
Assignee | ||
Comment 3•9 years ago
|
||
Screenshot for localizers
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → sarentz
Status: NEW → ASSIGNED
Assignee | ||
Comment 4•9 years ago
|
||
Attachment #8629573 -
Attachment is obsolete: true
Attachment #8629573 -
Flags: review?(sarentz)
Assignee | ||
Updated•9 years ago
|
Attachment #8631132 -
Flags: review?(sleroux)
Comment 5•9 years ago
|
||
Comment on attachment 8631132 [details] [review] PR: https://github.com/mozilla/firefox-ios/pull/689 Looks good to me - just left a comment about checking to see if we get an error back from the UIImageWriteToSavedPhotosAlbum method.
Attachment #8631132 -
Flags: review?(sleroux) → review+
Assignee | ||
Updated•9 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: noteworthy
Comment 6•9 years ago
|
||
Is camera roll still present in current iOS?
Updated•9 years ago
|
Flags: needinfo?(sarentz)
Assignee | ||
Comment 7•9 years ago
|
||
Yes, Camera Roll is the 'global' Album that contains all photos.
Flags: needinfo?(sarentz)
Comment 8•9 years ago
|
||
If you use iCloud Photo Library there seems to be no Camera Roll album at all, there is All Photos album instead - maybe we could reword this to be more general and handle both cases?
You need to log in
before you can comment on or make changes to this bug.
Description
•