Skip to content

Conversation

@microbit-matt-hillsdon
Copy link
Contributor

@microbit-matt-hillsdon microbit-matt-hillsdon commented Feb 21, 2025

The enter on a block toast shows generic help and we might consider showing it earlier if e.g. the user tabs to the workspace or otherwise focusses it by keyboard.

This logically conflicts with #458 which moves some related text to Blockly.Msg but (obviously!) not the new text. We can merge in either order and clean up after or block one on the other.

@rachel-fenichel
Copy link
Collaborator

@bmxedd do you have UX thoughts on best practices for toasts, particularly in this accessibility context?

@microbit-matt-hillsdon
Copy link
Contributor Author

Rebased on #408 for now.

@microbit-robert
Copy link
Contributor

Rebased on main and now uses the new Toast class in Blockly core.

Used in the following cases:

  • Copy (cleared on paste)
  • Cut (cleared on paste)
  • Enter on a block (cleared on toggling help)
  • First instance of inserting a block from the flyout with no constrained move positions
  • First instance of inserting a block from the flyout with constrained move positions
  • Attempting to move a block in constrained move mode when there are no constrained move positions

The enter on a block toast shows generic help and we might consider showing it
earlier if e.g. the user tabs to the workspace or otherwise focusses it by
keyboard.
@microbit-matt-hillsdon microbit-matt-hillsdon changed the title Illustrative toast feat: toasts for move/cut/copy and enter on a block Apr 24, 2025
@microbit-matt-hillsdon microbit-matt-hillsdon marked this pull request as ready for review April 24, 2025 14:45
@microbit-matt-hillsdon microbit-matt-hillsdon requested a review from a team as a code owner April 24, 2025 14:45
@microbit-matt-hillsdon microbit-matt-hillsdon requested review from maribethb and removed request for a team April 24, 2025 14:45
@rachel-fenichel rachel-fenichel removed the request for review from maribethb April 24, 2025 17:19
@gonfunko gonfunko merged commit 47d67fb into RaspberryPiFoundation:main Apr 25, 2025
6 of 8 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.

4 participants