Skip to content
This repository has been archived by the owner on Feb 3, 2022. It is now read-only.

Keyboard navigation #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Keyboard navigation #31

wants to merge 1 commit into from

Conversation

boxed
Copy link

@boxed boxed commented Apr 6, 2014

This is a first rough draft for keyboard navigation support. I'd like some feedback on what kind of changes you'd suggest before merging.

I'd like to eventually incorporate this into my media center (https://github.com/boxed/cmi), where remote control navigation is mapped to keyboard operations. But I think something like this would be useful to other people too.

@ovidiuch
Copy link
Contributor

ovidiuch commented Apr 8, 2014

Hi @boxed, the keyboard nav is pretty cool :)

Wondering what other use cases are there for this, or what would be an abstract behavior. You'd have to map some sort of "enter" event on a focus element, to be transmitted to the contained widget. It gets a bit complicated when you have other actions inside each widget (like 1/2/3x resize in our demo), because you'd have to pass along the focus to another level.

Definitely have to try your media center, XBMC started giving me headaches :)

@boxed
Copy link
Author

boxed commented Apr 8, 2014

Thanks!

In the case of CMi I just have big tiles that have a single action on them, so it's not a problem in my case. In the demo case I think it's more reasonable to map other keyboard shortcuts to the other commands. So for example '1' to 1x, '2' to 2x and '3' to 3x.

@mgifford
Copy link

What is the status of this? It's now a pretty old PR. Still keyboard only support should be considered.

@boxed
Copy link
Author

boxed commented Dec 16, 2017

I’d also like to know if this will be merged. I would like to be able to remove my fork if nothing else. It’s cluttering up my profile :P

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants