Skip to content

Conversation

@maribethb
Copy link
Collaborator

Note: this is against the add-screen-reader-support-experimental branch.

Removes a ts-expect-error declaration because in the corresponding branch of blockly, that property is public. The unnecessary expect causes tests to fail to compile.

@maribethb maribethb requested a review from a team as a code owner September 15, 2025 22:14
@maribethb maribethb requested review from BenHenning and removed request for a team September 15, 2025 22:14
@maribethb
Copy link
Collaborator Author

Unfortunately the rest of the test failures are because blockly is currently broken in this branch for renderers other than zelos. Prior to merging main into this branch, there were no tests in this repo that tested other renderers, and apparently the tests in core did not catch the fact that dragging blocks is broken (in non-zelos renderers), or perhaps the browser tests have not been run on this branch.

@maribethb
Copy link
Collaborator Author

Filed RaspberryPiFoundation/blockly#9367

I think it's useful to go ahead and merge this despite the remaining failures since the problem is in core, not this repo, tests will automatically pass again when that issue is fixed.

@maribethb
Copy link
Collaborator Author

The problems with geras/thrasos were fixed in core so this now passes.

@maribethb maribethb merged commit 59d4e8a into RaspberryPiFoundation:add-screen-reader-support-experimental Sep 16, 2025
8 of 10 checks passed
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.

2 participants