Closed Bug 1270399 Opened 8 years ago Closed 8 years ago

[TV][Home] Navigation in filter doesn't support to wrap around in both directions

Categories

(Firefox OS Graveyard :: Gaia::TV::Home, defect, P1)

ARM
Gonk (Firefox OS)
defect

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=0BwbmH5cX2W6Od0EwMXpSWlFwcDQ), if user navigates in filter, it should wrap around in both directions

[Testing Steps]
1. Navigation to filter
2. Move to left most filter then press left arrow key again
3. Move to right most filter then press right arrow key again

[Expected result]
Navigation should support to wrap around in both directions

[Actual result]
Navigation doesn't support to wrap around in both directions

[Build Info]
Environment: 2.6 Simulator
gaia:
https://github.com/mozilla-b2g/gaia/commit/4698bb1d24fac0db76f1f16f144cf0dae4bc3c23
gecko:
https://github.com/mozilla-b2g/gecko-b2g/commit/ccf59a9dda868db58b5d943e2c4235ead89f9344
Blocks: 1254038
blocking-b2g: 2.6? → 2.6+
Flags: needinfo?(lchang)
Assignee: nobody → lchang
Status: NEW → ASSIGNED
Flags: needinfo?(lchang)
Comment on attachment 8751627 [details] [review]
[gaia] luke-chang:1270399_tv_filter_wrap_around > mozilla-b2g:v2.6

Hi Rex,

In order to fulfill future requests, I added a new API to SpatialNavigator to support customized navigating rules via data attributes. Could you take a look? Thanks.
Attachment #8751627 - Flags: review?(rexboy)
Comment on attachment 8751627 [details] [review]
[gaia] luke-chang:1270399_tv_filter_wrap_around > mozilla-b2g:v2.6

awesome!
Attachment #8751627 - Flags: review?(rexboy) → review+
Rex, Thanks a lot!


landed on v2.6: https://github.com/mozilla-b2g/gaia/commit/257e64ab5b98fbf78e846fce74355015c4452489
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: