-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FocusManager: enable more keyboard shortcuts by default #11871
Conversation
Also add ContextMenu for hold.
I think at the very least tab/shift+tab and a means to hold should just be there by default. They're not used for anything. :-) |
Sym+AA potentially sounds a little weird since they're both modifiers? The rest all works great and isn't used anywhere for anything else, except for tab/shift+tab for selecting links, but that doesn't conflict with any widgets that come on top. Also pinging @Commodore64user for thoughts on Sym+AA. |
If "Sym+Aa" is supposed to be Hold (long press) then, first it doesn't quite work and second, it doesn't seem like a good combination either: i would have "Shift+Press" or "Alt+Press" instead (or both). Tab keys don't exist on kindle so that would not apply here. I don't think Aa is a modifier though' don't quote me. Having said that, there are a lot of other quirks on kindles with keyboards, but running KOReader on these older models is quite painful (much to my chagrin) that i am not sure it's worth going out of one's way to fix them. |
I thought I saw AA in the list when you added ScreenKB to modifiers but it doesn't seem to be there.
Those are available on both types of device?
Curious, Sym+Tab is an explicitly supported keyboard shortcut as equivalent to Shift+Tab. I wonder what that's for.
What's good for keyboard Kindle is good for PC keyboard is good for gamepad is good for Bluetooth keyboard is good for TV remote. |
I think "Sym" shouldn't even be one. Sym should just call the virtual keyboard when one needs weird symbols and that sort of thing.
They [Shift and Alt keys] are both available on kindles with keyboards, not on K4 (that's why i went with ScreenKB+Press there). Currently not assigned to anything according to my tests.
I am just as curious now, i thought maybe the older K2s and DX models maybe had one [Tab key] but a quick search proves otherwise https://duckduckgo.com/?q=kindle+2&iar=images&iax=images&ia=images&iai=http%3A%2F%2Fwww.gearlive.com%2Fimages%2Fgearlive-gallery%2F020-kindle-2-unboxing.jpg so not sure what that's about
Besides @pazos who else is reading on a TV? Crazy. |
Right shift is defined as Sym in SDL, probably to be able to test it. Emulator and normal usage might be at odds for that one… but Sym + Tab is also literally the only place it's used for anything at all. |
Could you please add long press on kk's here with shift + press (so i don't have to)? |
excellent point. best to keep both then |
Also add ContextMenu for hold.
@comphilip I don't see much of a reason not to enable these, unless there's a specific reason not to?
This change is