Skip to content
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

Merged
merged 4 commits into from
May 23, 2024

Conversation

Frenzie
Copy link
Member

@Frenzie Frenzie commented May 21, 2024

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 Reviewable

@comphilip
Copy link
Contributor

@Frenzie Disable these to void breaking compatibility without testing. Letting advanced user to turn it on via settings.reader.lua. I posted Kindle DX and PC keyboard example setting in #8859

@Frenzie
Copy link
Member Author

Frenzie commented May 22, 2024

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. :-)

@Frenzie
Copy link
Member Author

Frenzie commented May 22, 2024

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.

Frenzie added a commit to koreader/koreader-base that referenced this pull request May 22, 2024
@Commodore64user
Copy link
Contributor

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.

@Frenzie
Copy link
Member Author

Frenzie commented May 22, 2024

I thought I saw AA in the list when you added ScreenKB to modifiers but it doesn't seem to be there.

i would have "Shift+Press" or "Alt+Press" instead (or both).

Those are available on both types of device?

Tab keys don't exist on kindle so that would not apply here.

Curious, Sym+Tab is an explicitly supported keyboard shortcut as equivalent to Shift+Tab. I wonder what that's for.

i am not sure it's worth going out of one's way to fix them.

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.

@Commodore64user
Copy link
Contributor

I thought I saw AA in the list when you added ScreenKB to modifiers but it doesn't seem to be there.

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.

i would have "Shift+Press" or "Alt+Press" instead (or both).

Those are available on both types of device?

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.

Curious, Sym+Tab is an explicitly supported keyboard shortcut as equivalent to Shift+Tab. I wonder what that's for.

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

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.

Besides @pazos who else is reading on a TV? Crazy.

@Frenzie
Copy link
Member Author

Frenzie commented May 22, 2024

I am just as curious now

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.

@Commodore64user
Copy link
Contributor

Commodore64user commented May 22, 2024

Could you please add long press on kk's here with shift + press (so i don't have to)?

@comphilip
Copy link
Contributor

comphilip commented May 23, 2024

@Frenzie

Sym+AA potentially sounds a little weird since they're both modifiers?

Tested in Kindle DX, Sym is modifier while AA is not.

Kindle DX joystick
Kindle DX uses joystick for arraw and press. The UX is worse than buttons and easy to misoperation.

So I choose Sym+AA

@Commodore64user
Copy link
Contributor

excellent point. best to keep both then Sym+Aa and Shift+Press

@Frenzie Frenzie merged commit 20d8a5c into koreader:master May 23, 2024
2 checks passed
@Frenzie Frenzie deleted the focusmanager-keys branch May 23, 2024 05:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants