Closed
Bug 1021284
Opened 10 years ago
Closed 10 years ago
[camera] [flame] spinner appears after every photo capture
Categories
(Firefox OS Graveyard :: Gaia::Camera, defect)
Tracking
(b2g-v2.0 affected)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
b2g-v2.0 | --- | affected |
People
(Reporter: tif, Unassigned)
Details
(Whiteboard: interaction-design)
Attachments
(2 files)
STR - on the Flame: 1. Open Camera app 2. Take a photo Expected Spinner shows for significant delays Actual Spinner shows for every photo
Reporter | ||
Updated•10 years ago
|
Whiteboard: interaction-design
Comment 1•10 years ago
|
||
Attachment #8435754 -
Flags: ui-review?(tshakespeare)
Attachment #8435754 -
Flags: review?(jdarcangelo)
Comment 2•10 years ago
|
||
Comment on attachment 8435754 [details] [review] pull-request (master) Going to R- until you make the changes we discussed. Flag me again when ready, thanks!
Attachment #8435754 -
Flags: review?(jdarcangelo) → review-
Reporter | ||
Comment 3•10 years ago
|
||
Comment on attachment 8435754 [details] [review] pull-request (master) Excellent! Spinner shows up for HDR but not after every single photo. Thanks Wilson!
Reporter | ||
Comment 4•10 years ago
|
||
Comment on attachment 8435754 [details] [review] pull-request (master) Actually changing the flag this time! :)
Attachment #8435754 -
Flags: ui-review?(tshakespeare) → ui-review+
Comment 6•10 years ago
|
||
Comment on attachment 8435754 [details] [review] pull-request (master) Implemented discussed more useful solution. If the busy 'type' is defined in the loadScreen config, we display a loading screen after the given delay. For unrecognised or undefined types, no loading screen is shown.
Attachment #8435754 -
Flags: review- → review?(jdarcangelo)
Flags: needinfo?(wilsonpage)
Comment 7•10 years ago
|
||
Comment on attachment 8435754 [details] [review] pull-request (master) Looks good man! Thanks for making the changes!
Attachment #8435754 -
Flags: review?(jdarcangelo) → review+
Comment 8•10 years ago
|
||
Landed on 'master' https://github.com/mozilla-b2g/gaia/commit/a0f9f1f41a436daad8a249ce85df80a81a5ba2d5
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 9•10 years ago
|
||
This issue also occurs on today's 2.0 Flame. Environmental Variables: Device: Flame 2.0 Build ID: 20140701000201 Gaia: 8fb5e2a9ad1025ee7d247b90af7499766afadd28 Gecko: 5da69a493324 Version: 32.0a2 (2.0) Firmware Version: v122 User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0
status-b2g-v2.0:
--- → affected
Flags: needinfo?(wilsonpage)
Comment 11•10 years ago
|
||
Oops ignore last comment, wrong bug.
Comment 12•10 years ago
|
||
[flame][v2.0] Everything seems to work as expected. - Taking pictures with HDR *off*, shows no spinner. - Taking pictures with HDR *on*, shows spinner to account for processing time.
Comment 13•10 years ago
|
||
[flame][v2.0] CORRECTION: I am able to reproduce.
Comment 14•10 years ago
|
||
Re-opening to address v2.0 uplift.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 15•10 years ago
|
||
Attachment #8449503 -
Flags: review?(jdarcangelo)
Comment 16•10 years ago
|
||
Comment on attachment 8449503 [details] [review] pull-request (v2.0) LGTM!
Attachment #8449503 -
Flags: review?(jdarcangelo) → review+
Comment 17•10 years ago
|
||
Landed on 'v2.0' https://github.com/mozilla-b2g/gaia/commit/28247f02465c7025d455257242ba5f98c1bcd009
Updated•10 years ago
|
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•