Skip to content

[Hotkeys] remove alphabetic hotkeys from K4#13950

Merged
Frenzie merged 1 commit into
koreader:masterfrom
Commodore64user:hotkeys-defaults
Jun 21, 2025
Merged

[Hotkeys] remove alphabetic hotkeys from K4#13950
Frenzie merged 1 commit into
koreader:masterfrom
Commodore64user:hotkeys-defaults

Conversation

@Commodore64user

@Commodore64user Commodore64user commented Jun 14, 2025

Copy link
Copy Markdown
Member

what's new

  • Updated defaults to check for the presence of a keyboard before enabling their functionalities (dictionary_lookup, file_search, wikipedia_lookup, and fulltext_search).

This change is Reviewable

@Commodore64user Commodore64user changed the title [Hotkeys] remove alphabet based hotkeys from K4 [Hotkeys] remove alphabetic hotkeys from K4 Jun 14, 2025
@Frenzie

Frenzie commented Jun 14, 2025

Copy link
Copy Markdown
Member

That's possibly a bit too smart (or not smart enough), saving a few bytes at the expense of e.g. BT keyboards?

@Commodore64user

Commodore64user commented Jun 14, 2025

Copy link
Copy Markdown
Member Author

If a device doesn't have a keyboard, hotkeys will not run therefore defaults will not be set. They should only run once there is a keyboard connected thus, no problem?

@Frenzie

Frenzie commented Jun 14, 2025

Copy link
Copy Markdown
Member

There's indeed no problem if it runs once connected.

@Commodore64user

Copy link
Copy Markdown
Member Author

Since you’re in such merging spirit, would you have a look at all these little ones @Frenzie

@Frenzie Frenzie merged commit 306fd6c into koreader:master Jun 21, 2025
2 of 4 checks passed
@Frenzie Frenzie added this to the 2025.06 milestone Jun 21, 2025
@Commodore64user Commodore64user deleted the hotkeys-defaults branch June 21, 2025 09:27
0xstillb pushed a commit to 0xstillb/koreader-thai that referenced this pull request May 9, 2026
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.

2 participants