Skip to content

Conversation

@RoboErikG
Copy link
Contributor

The basics

The details

Resolves

Fixes RaspberryPiFoundation/blockly-keyboard-experimentation#258

Proposed Changes

Remove the alt+key code version of commands

Reason for Changes

Using alt is non-standard and may conflict with window commands, so we're removing it.

Test Coverage

Removed the tests that expect alt to work.

Documentation

Additional Information

@RoboErikG RoboErikG added issue: bug Describes why the code or behaviour is wrong breaking change Used to mark a PR or issue that changes our public APIs. labels May 2, 2025
@RoboErikG RoboErikG requested a review from a team as a code owner May 2, 2025 17:06
@RoboErikG RoboErikG requested a review from BenHenning May 2, 2025 17:06
Copy link
Collaborator

@rachel-fenichel rachel-fenichel left a comment

Choose a reason for hiding this comment

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

Change the pr title to fix! to indicate that this is a breaking change for release notes purposes.

@RoboErikG RoboErikG changed the title fix: Remove alt+key commands fix!: Remove alt+key commands May 2, 2025
@RoboErikG RoboErikG merged commit a4e6166 into RaspberryPiFoundation:rc/v12.0.0 May 2, 2025
10 of 12 checks passed
@RoboErikG RoboErikG deleted the no-alt branch June 23, 2025 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change Used to mark a PR or issue that changes our public APIs. issue: bug Describes why the code or behaviour is wrong

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants