Closed
Bug 822628
Opened 13 years ago
Closed 13 years ago
Does not remember analogical/digital display after closing the clock.
Categories
(Firefox OS Graveyard :: Gaia::Clock, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: nbp, Assigned: janjongboom)
Details
(Keywords: b2g-testdriver, Whiteboard: polish, UX-P1, TEF_REQ, PRODUCT-FIXFEATURE)
Attachments
(1 file)
|
46 bytes,
patch
|
iliu
:
review+
|
Details | Diff | Splinter Review |
Device:
Version 1.0.0
Platform 18.0
Build id 20121206062128
STR:
- Open the clock
- tap on the clock
* The display switch from analogical to digital
- Press Home
- Swipe up the clock app to close
- Open the clock
Observed:
The outdated screenshot showing the digital clock is shown and is replaced by the analogical clock.
Expected:
The digital clock is shown (screenshot) and updated as soon as the app finish restarting.
Comment 1•13 years ago
|
||
I think it's not a blocking issue. But we can remember user's latest display mode in the indexedDB.
agree with Ian, this is a polish but also making sense to remember the latest mode since we have mode switching functionality
Ian, can you ask for approving on this?
Whiteboard: polish
| Assignee | ||
Comment 3•13 years ago
|
||
Here's a patch: https://github.com/mozilla-b2g/gaia/pull/7708
Comment 4•13 years ago
|
||
(In reply to janjongboom from comment #3)
> Here's a patch: https://github.com/mozilla-b2g/gaia/pull/7708
Hi Jan,
Thanks for your help. And I quickly review your patch with some comment on github.
Updated•13 years ago
|
QA Contact: jshih → fyen
| Assignee | ||
Updated•13 years ago
|
Assignee: nobody → janjongboom
| Assignee | ||
Comment 5•13 years ago
|
||
Attachment #706280 -
Flags: review?(iliu)
| Assignee | ||
Updated•13 years ago
|
Status: NEW → ASSIGNED
Comment 6•13 years ago
|
||
Comment on attachment 706280 [details] [diff] [review]
Patch for this issue
Jan,
Since you revised all comments according my suggestion, the patch works for me.
r=me
Thanks for your help.
Nice job!
Attachment #706280 -
Flags: review?(iliu) → review+
Updated•13 years ago
|
Whiteboard: polish → polish, UX-P1, TEF_REQ
Comment 7•13 years ago
|
||
Jan,
Since the pr(https://github.com/mozilla-b2g/gaia/pull/7708) is merged, you can close the issue.
Thanks.
| Assignee | ||
Updated•13 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
Whiteboard: polish, UX-P1, TEF_REQ → polish, UX-P1, TEF_REQ, PRODUCT-FIXFEATURE
Comment 8•13 years ago
|
||
Device unagi
gaia 122ee16093d9730de7a9c6beba989adeb5d716ae
gecko 95bf443385bbd8ee611be1586ec975e609acd006
Verified.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•