Skip to content

fix(web): call postKeystroke on banner touch 🕊#6004

Merged
mcdurdin merged 3 commits intomasterfrom
fix/web/3621-call-postkeystroke-on-banner-click
Jan 17, 2022
Merged

fix(web): call postKeystroke on banner touch 🕊#6004
mcdurdin merged 3 commits intomasterfrom
fix/web/3621-call-postkeystroke-on-banner-click

Conversation

@mcdurdin
Copy link
Copy Markdown
Member

@mcdurdin mcdurdin commented Dec 3, 2021

Relates to #3621.

Merges into #5963.

Ensures that the begin postKeystroke entry point is called when the user accepts a suggestion.

@keymanapp-test-bot skip

We'll do testing separately later on this.

Relates to #3621.

Ensures that the `begin postKeystroke` entry point is called when the
user accepts a suggestion.
@mcdurdin mcdurdin added this to the A15S19 milestone Dec 3, 2021
@mcdurdin mcdurdin changed the title fix(web): call postKeystroke on banner touch fix(web): call postKeystroke on banner touch 🍆 Dec 3, 2021
@keymanapp-test-bot
Copy link
Copy Markdown

keymanapp-test-bot bot commented Dec 3, 2021

Copy link
Copy Markdown
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 noticed a typo. I think Darcy is probably best to give the final tick of approval.

Copy link
Copy Markdown
Contributor

@darcywong00 darcywong00 left a comment

Choose a reason for hiding this comment

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

lgtm

Co-authored-by: rc-swag <58423624+rc-swag@users.noreply.github.com>
@mcdurdin mcdurdin changed the title fix(web): call postKeystroke on banner touch 🍆 fix(web): call postKeystroke on banner touch 🕊 Dec 9, 2021
@mcdurdin mcdurdin modified the milestones: A15S19, A15S20 Dec 11, 2021
@mcdurdin mcdurdin modified the milestones: A15S20, A15S21, A15S22 Jan 3, 2022
Base automatically changed from feat/web/3621-newcontext-group to master January 17, 2022 01:04
@mcdurdin mcdurdin merged commit 1b57c11 into master Jan 17, 2022
@mcdurdin mcdurdin deleted the fix/web/3621-call-postkeystroke-on-banner-click branch January 17, 2022 01:14
@keyman-server
Copy link
Copy Markdown
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants