Closed
Bug 1006348
Opened 11 years ago
Closed 11 years ago
[Cost control] After minimizing the app with the settings screen opened, this view is closed when the app is relaunched.
Categories
(Firefox OS Graveyard :: Gaia::Cost Control, defect)
Tracking
(blocking-b2g:1.3+, b2g-v1.3 fixed, b2g-v1.3T fixed, b2g-v1.4 fixed, b2g-v2.0 fixed)
People
(Reporter: mai, Assigned: mai)
References
Details
Attachments
(1 file)
|
46 bytes,
text/x-github-pull-request
|
jmcf
:
review+
bajaj
:
approval-gaia-v1.3+
|
Details | Review |
The app does not maintain the previous view when it is relaunched.
| Assignee | ||
Updated•11 years ago
|
Assignee: nobody → mri
| Assignee | ||
Comment 1•11 years ago
|
||
Jose,
Could you review the patch?
Regards
Attachment #8417913 -
Flags: review?(jmcf)
Comment 2•11 years ago
|
||
Comment on attachment 8417913 [details] [review]
patch v1.0
works perfectly.
thanks
Attachment #8417913 -
Flags: review?(jmcf) → review+
| Assignee | ||
Comment 3•11 years ago
|
||
Master: a3abb00ae7820aed4785f285b8fe7388df06b985
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
status-b2g-v2.0:
--- → fixed
Target Milestone: --- → 2.0 S1 (9may)
Comment 5•11 years ago
|
||
Vance - Can you find out if this is a cert blocker for TCL?
Flags: needinfo?(vchen)
Yes, this is a cert blocker, thanks
Flags: needinfo?(vchen)
Comment 7•11 years ago
|
||
Per comment 6, setting blocking-b2g flag accordingly
blocking-b2g: --- → 1.3?
Updated•11 years ago
|
blocking-b2g: 1.3? → 1.3+
| Assignee | ||
Comment 8•11 years ago
|
||
Comment on attachment 8417913 [details] [review]
patch v1.0
NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings.
Checking the view id before closing the dialog after when a visibility_change event is fired.
[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): 963682
[User impact] if declined: It's not possible open the settings screen after minimizing the app.
[Testing completed]: Testing on device
[Risk to taking this patch] (and alternatives if risky): Low risk
[String changes made]: No
Attachment #8417913 -
Flags: approval-gaia-v1.3?(release-mgmt)
Comment 9•11 years ago
|
||
Updated•11 years ago
|
Attachment #8417913 -
Flags: approval-gaia-v1.3?(release-mgmt) → approval-gaia-v1.3+
Comment 10•11 years ago
|
||
Updated•11 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•