Closed Bug 881664 Opened 6 years ago Closed 6 years ago
Camera 'pick' pitfall when there is no sdcard
17.54 KB, image/png
88 bytes, text/html
59.41 KB, image/png
60.19 KB, image/png
If you try to take a picture from the Camera when there is no SD card and that this is started from the 'pick' WebActivity, you get stuck in the "No SD card" error message with no other way to get out than to reboot. Steps to reproduce: 0. Ensure no SD card is present in the device 1. Open SMS app, tap to attach a file 2. From the activity chooser, select Camera 3. Kaboom, you get suck on the error message stating that there is no SD card. Reproduced with gaia master.
Dale, I done the following change: 1. add a header section in overlay only containing back button. 2. add css of this header section to show above overlay-container 3. check if it is shown with pick activity, if yes, show header section, vice visa. 4. Once overlay shown, all buttons are disabled including cancelPick. So, I create another method "cancelPickActivity" which is chained with cancelPick function for back button.
Attachment #762475 - Flags: review?(dale)
Comment on attachment 762475 [details] patch for this bug I am going to clear the review and needinfo UX for this, the code looks great and I think it would be mergable as it, but not sure about the UX I feel like this should probably be an alert style UI with a big red button in the bottom to cancel the pick activity, the small header button on top of an overlay feels wrong and non obvious for the user what to do.
Attachment #762475 - Flags: review?(dale) → feedback?(firefoxos-ux-bugzilla)
I've noticed the same behavior with Music, and probably with Videos too ...
Thanks Dale. Let's wait for UX's opinion.
Hi Stephany, can you help with this? its been pending for some time. Thanks
Wayne, it's been "pending for some time" but only because UX was flagged for the first time on July 4, a national holiday. Today is our first day back since we also had July 5 off for the FFOS launch. I will assign it to someone on our team this morning because it is blocking.
Flags: needinfo?(swilkes) → needinfo?(firefoxos-ux-bugzilla)
The feedback for firstname.lastname@example.org was filed on the 14th of June, but I marked feedback? instead of needinfo? apologies if that is the wrong way
Ah - good to know. This wouldn't have come up in our usual query because feedback is only attached to the patch. Sorry about that. Assigned to Rob.
Flags: needinfo?(firefoxos-ux-bugzilla) → needinfo?(rmacdonald)
Comment on attachment 762475 [details] patch for this bug Clearing f? as should have been needinfo
Attachment #762475 - Flags: feedback?(firefoxos-ux-bugzilla)
In this case, I propose we eliminate the need to show the error by removing pick options that are not available. So, if the memory card is not present, Gallery and Camera, for example, would not appear on the list. I'm assuming here that the Wallpaper link will always be available as the images are stored on phone memory. Please flag email@example.com if you have any questions or concerns.
Dale, Is it possible to implement like that?? The activity list is provided by others app which may not have the knowledge of availability checking of camera, or gallery app, for example. And there is also a case about plugging USB cable while processing 'pick' activity.
Yup, that would require the system app having knowledge of when particular applications were able to provide these activities, I dont think that is really possible within the current implementation of activities (and cant think of any clean way to extend them to be able to do so), and as you brought up it is always possible that the sdcard becomes inaccessible during picking an image. re needinfoing firefox-ux as I am fairly sure we will need to implement this as was originally patched (just with a possible improved UI)
Flags: needinfo?(dale) → needinfo?(firefoxos-ux-bugzilla)
Thanks Dale. What's the UI question for the latest needinfo? Is it a question of what should be implemented, something else? Sorry I'm not grokking! :(
Hi John and Dale... I had a feeling that proposal might not be feasible within the timeframe. That said, I'd still like to work towards that and have flagged Peter to add it to the systems group's long backlog. As an interim solution, I've attached a screenshot that is more in line with our UX guidelines. Please flag me if you have any questions or concerns. Best regards, Rob
Flags: needinfo?(firefoxos-ux-bugzilla) → needinfo?(pdolanjski)
!. use building block confirm to display the error message 2. remove css which is duplicated with building block 3. add codes to show close button when picking.
Comment on attachment 775550 [details] use building block to show the error overlay Looks like a good patch, much nicer to be using building blocks There is a syntax error that prevents it from working, mentioned on Github, there is also an wording issue as the error message doesnt make sense, but that is unrelated to this patch.
Attachment #775550 - Flags: review?(dale) → review-
Comment on attachment 775550 [details] use building block to show the error overlay Sorry about the syntax error. That should be that. It may be some keyboard typo here. I had fix the "comma" error and rerun the whole test. Please review it again. Thanks.
Attachment #775550 - Flags: review- → review?(dale)
Comment on attachment 775550 [details] use building block to show the error overlay This is great, thanks
Attachment #775550 - Flags: review?(dale) → review+
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
I was not able to uplift this bug to v1-train. If this bug has dependencies which are not marked in this bug, please comment on this bug. If this bug depends on patches that aren't approved for v1-train, we need to re-evaluate the approval. Otherwise, if this is just a merge conflict, you might be able to resolve it with: git checkout v1-train git cherry-pick -x -m1 cdf71136ae6233cb790e9f9be16ae456726e38c1 <RESOLVE MERGE CONFLICTS> git commit
Hi John Ford, It is just a merge conflict. I had create a branch in my github repository for this bug in v1-train: https://github.com/huchengtw-moz/gaia/tree/bug_881664-v1-train Thanks for your help.
(In reply to Rob MacDonald [:robmac] from comment #15) > That said, I'd still like to work towards that and have flagged Peter to add > it to the systems group's long backlog. Added to backlog.
No error message appears stating that there is no sd card inserted Build ID: 20130722070207 Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/68fb0a2e0114 Gaia: 41d10fb10be6916e6554eb440d9a97130ef23ce0 Platform Version: 18.1
You need to log in before you can comment on or make changes to this bug.