Closed
Bug 1282703
Opened 8 years ago
Closed 8 years ago
[TV][Home] Unpin card shouldn't show confirmation page
Categories
(Firefox OS Graveyard :: Gaia::TV::Home, defect, P1)
Tracking
(blocking-b2g:2.6?)
VERIFIED
FIXED
blocking-b2g | 2.6? |
People
(Reporter: mlien, Assigned: lchang)
References
Details
(Whiteboard: [ft:conndevices])
Attachments
(1 file)
[Description] According to the latest UX spec(https://drive.google.com/open?id=0BwbmH5cX2W6OVnhuU2p1bTZPVGc) page 21, unpin card should remove card from home without confirmation page [Testing Steps] 1. Pin any app to home 2. Home -> Navigation to step 1's app 3. Press option key -> Choose unpin from home -> OK [Expected result] Should unpin app from home directly [Actual result] Show confirmation page [Build Info] Environment: 2.6 Simulator gaia: https://github.com/mozilla-b2g/gaia/commit/500eba02e3bf90099eb23864008f3f2ae3bb4947 gecko: https://github.com/mozilla-b2g/gecko-b2g/commit/49b98aa3b415380cc7a021d740b242894654245a
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → lchang
Status: NEW → ASSIGNED
Comment 1•8 years ago
|
||
Assignee | ||
Comment 2•8 years ago
|
||
Comment on attachment 8765809 [details]
[gaia] luke-chang:1282703_remove_unpin_confirmation > mozilla-b2g:v2.6
Hi Rex,
It's a simple patch. Thanks.
Attachment #8765809 -
Flags: review?(rexboy)
Comment 3•8 years ago
|
||
Comment on attachment 8765809 [details]
[gaia] luke-chang:1282703_remove_unpin_confirmation > mozilla-b2g:v2.6
Looks good to me.
Attachment #8765809 -
Flags: review?(rexboy) → review+
Assignee | ||
Comment 4•8 years ago
|
||
Rex, Thanks a lot. landed on v2.6: https://github.com/mozilla-b2g/gaia/commit/efa9514dbd028bbd243246cb94e354d78ccff04b
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 5•8 years ago
|
||
verified and fixed with the latest build gaia: https://github.com/mozilla-b2g/gaia/commit/efa9514dbd028bbd243246cb94e354d78ccff04b gecko: https://github.com/mozilla-b2g/gecko-b2g/commit/e81fd19f9e908939f9476b95b85d09e155b4aaea
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•