Skip to content

TINYMCE-14795: Fix mouse reset code - #171

Merged
TheSpyder merged 3 commits into
masterfrom
feature/TINYMCE-14795
Aug 25, 2026
Merged

TINYMCE-14795: Fix mouse reset code#171
TheSpyder merged 3 commits into
masterfrom
feature/TINYMCE-14795

Conversation

@TheSpyder

Copy link
Copy Markdown
Member

Related Ticket: TINYMCE-14795

Description of Changes:

  • We've long had mouse reset code in bedrock, but it had two problems
    1. It was scoped to Chrome browsers
    2. It only activates on test start, so the change in 15.0.0 to reduce network traffic accidentally bypassed it
  • The code is now enabled for all browsers
  • On the client, a wrapper around XHR detects when mouse events are used (our test library doesn't use fetch for this) and sets a flag so the next test sends a start event to reset the mouse
  • to avoid this causing extra spam to the server, the 30s result pings are included in test starts now

This should fix our build failures on Firefox 153, which appears to have changed something about the mouse hover code.

Pre-checks:

  • Changelog entry added
  • package.json versions have not been changed (done by Lerna on release)
  • Tests have been added (if applicable)

Before merging:

  • Ensure internal dependencies are on appropriate versions
    • For stable releases, all dependencies must be stable
    • For release candidates, all dependencies must be release candidates or stable

@TheSpyder
TheSpyder requested a review from a team as a code owner August 21, 2026 07:20
Comment thread modules/runner/src/main/ts/reporter/Reporter.ts Outdated
Comment thread modules/server/src/main/ts/bedrock/server/Apis.ts Outdated
@TheSpyder
TheSpyder force-pushed the feature/TINYMCE-14795 branch from 3432cfa to d9707ba Compare August 25, 2026 01:38
@TheSpyder
TheSpyder force-pushed the feature/TINYMCE-14795 branch from d9707ba to 2ae39a9 Compare August 25, 2026 02:20
@TheSpyder
TheSpyder merged commit 69a6e7b into master Aug 25, 2026
4 checks passed
@TheSpyder
TheSpyder deleted the feature/TINYMCE-14795 branch August 25, 2026 02:27
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.

3 participants