Use textbox shortcuts from the host platform #52

Merged
MartinFouilleul merged 3 commits from ilidemi/orca:platform-text-edit into main 2023-08-22 20:15:22 +00:00
Collaborator

Fixes half of ☐4 from #30. Names are picked somewhat arbitrarily, open to feedback. Builds on !50, hopefully rebase/merge should clear that out. Invisible changes on comments are line endings, looks like some comments have CRLF committed.

Fixes half of ☐4 from #30. Names are picked somewhat arbitrarily, open to feedback. Builds on !50, hopefully rebase/merge should clear that out. Invisible changes on comments are line endings, looks like some comments have CRLF committed.
ilidemi added 1 commit 2023-08-22 08:13:26 +00:00
Collaborator

You can bind orca_get_host_platform() directly without having a forwarding oc_get_host_platform_impl().

Also, I think it's still useful to keep OC_KEYMOD_MAIN_MODIFIER around as a convenience for people who want to implement custom shortcut for their app, but don't want/need to deal with choosing which modifier to use depending on the platform.

You can bind `orca_get_host_platform()` directly without having a forwarding `oc_get_host_platform_impl()`. Also, I think it's still useful to keep `OC_KEYMOD_MAIN_MODIFIER` around as a convenience for people who want to implement custom shortcut for their app, but don't want/need to deal with choosing which modifier to use depending on the platform.
ilidemi added 2 commits 2023-08-22 16:43:51 +00:00
MartinFouilleul merged commit 2c07c9b665 into main 2023-08-22 20:15:22 +00:00
MartinFouilleul deleted branch platform-text-edit 2023-08-22 20:15:22 +00:00
Sign in to join this conversation.
No reviewers
No Label
macOS
windows
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: hmn/orca#52
No description provided.