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

feat(core): action struct to action items conversion #10115

Merged
merged 2 commits into from
Dec 12, 2023

Conversation

mcdurdin
Copy link
Member

@mcdurdin mcdurdin commented Dec 1, 2023

Relates to #9999.

Adds state->set_actions(). This sets the Core's action list to match the contents of the action struct. Note that markers are not supported and backspace expected_values will be empty, as this information is not available. As the intended consumer of the action struct does not need to know this information, this should be adequate.

@keymanapp-test-bot skip

Adds state->set_actions(). This sets the Core's action list to match the
contents of the action struct. Note that markers are not supported and
backspace expected_values will be empty, as this information is not
available. As the intended consumer of the action struct does not need
to know this information, this should be adequate.
@keymanapp-test-bot
Copy link

keymanapp-test-bot bot commented Dec 1, 2023

@keymanapp-test-bot keymanapp-test-bot bot added this to the A17S27 milestone Dec 1, 2023
@github-actions github-actions bot added core/ Keyman Core feat labels Dec 1, 2023
Copy link
Contributor

@rc-swag rc-swag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just few commetns

core/src/action.cpp Outdated Show resolved Hide resolved
core/src/action.cpp Show resolved Hide resolved
core/tests/unit/kmnkbd/action_set_api.cpp Show resolved Hide resolved
core/tests/unit/kmnkbd/action_set_api.cpp Show resolved Hide resolved
core/src/state.hpp Show resolved Hide resolved
@mcdurdin mcdurdin requested a review from srl295 December 5, 2023 02:21
@mcdurdin mcdurdin modified the milestones: A17S27, A17S28 Dec 8, 2023
Copy link
Contributor

@rc-swag rc-swag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated my comments
LGTM

core/src/action.cpp Show resolved Hide resolved
core/src/state.hpp Show resolved Hide resolved
@mcdurdin mcdurdin merged commit e65d5c5 into master Dec 12, 2023
17 checks passed
@mcdurdin mcdurdin deleted the feat/core/action-struct-to-queue branch December 12, 2023 21:45
@keyman-server
Copy link
Collaborator

Changes in this pull request will be available for download in Keyman version 17.0.229-alpha

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core/ Keyman Core feat
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants