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

refactor(web): move gesture-recognizer β†’ gesture-processor πŸ—οΈ #12194

Merged
merged 6 commits into from
Aug 29, 2024

Conversation

ermshiperete
Copy link
Contributor

@ermshiperete ermshiperete commented Aug 15, 2024

Moves common/web/gesture-recognizer/ β†’ web/src/engine/osk/gesture-processor.

Fixes: #12151

@keymanapp-test-bot skip

@keymanapp-test-bot keymanapp-test-bot bot added this to the A18S8 milestone Aug 15, 2024
@ermshiperete ermshiperete force-pushed the refactor/web/12148_worker-thread branch from 08e0d30 to ea62ac2 Compare August 16, 2024 06:35
@ermshiperete ermshiperete force-pushed the refactor/web/12151_gestures branch from 568c324 to 3e59a7c Compare August 16, 2024 17:47
@darcywong00 darcywong00 modified the milestones: A18S8, A18S9 Aug 17, 2024
Base automatically changed from refactor/web/12148_worker-thread to master August 19, 2024 08:10
@ermshiperete ermshiperete force-pushed the refactor/web/12151_gestures branch from 3e59a7c to 5435055 Compare August 19, 2024 08:13
ermshiperete and others added 5 commits August 26, 2024 15:29
Moves `common/web/gesture-recognizer/` β†’
`web/src/engine/osk/gesture-processor`.

Fixes: #12151
`gesture-processor/src/test/auto/browser/web-test-runner.config.mjs`
run the tests on iOS and Android. However, if we enable these browsers
in `web/src/test/auto/dom/web-test-runner.config.mjs` we get failing
tests in osk, therefore these browsers are commented for now.
`test.sh` got removed since it did the same as `build.sh test`.
Previously running the tests failed because it couldn't find ICU 70 when
running webkit. Ubuntu 24.04 Noble comes with ICU 74. Updating
playwright to 1.46 fixes this.
@ermshiperete ermshiperete force-pushed the refactor/web/12151_gestures branch from 1c2e537 to e788bf5 Compare August 26, 2024 09:26
@ermshiperete ermshiperete marked this pull request as ready for review August 28, 2024 07:02
@ermshiperete ermshiperete merged commit 39f0d5b into master Aug 29, 2024
21 checks passed
@ermshiperete ermshiperete deleted the refactor/web/12151_gestures branch August 29, 2024 02:16
@keyman-server
Copy link
Collaborator

Changes in this pull request will be available for download in Keyman version 18.0.101-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.

refactor(web): common/web/gesture-recognizer/ β†’ web/src/engine/osk/gesture-processor
4 participants