Skip to content

fix: Improve navigation and movement looping behavior#9732

Merged
gonfunko merged 9 commits intov13from
froot-loops
Apr 20, 2026
Merged

fix: Improve navigation and movement looping behavior#9732
gonfunko merged 9 commits intov13from
froot-loops

Conversation

@gonfunko
Copy link
Copy Markdown
Contributor

The basics

The details

Resolves

Proposed Changes

This PR makes several improvements around keyboard-driven movement and navigation:

  • Looping now defaults to on
  • The prompt to try unconstrained move mode when a block is moved to the workspace as a top-level block in constrained mode is now only shown once per session
  • When blocks are moved to top-level blocks during constrained mode, their position is offset from their prior position to make it clearer that the proposed drop location/state has changed
  • Constrained move mode now respects the keyboard navigation looping setting

@gonfunko gonfunko requested a review from a team as a code owner April 17, 2026 20:32
@gonfunko gonfunko requested a review from maribethb April 17, 2026 20:32
@gonfunko gonfunko changed the title Froot loops fix: Improve navigation and movement looping behavior Apr 17, 2026
@github-actions github-actions Bot added the PR: fix Fixes a bug label Apr 17, 2026
Copy link
Copy Markdown
Contributor

@maribethb maribethb left a comment

Choose a reason for hiding this comment

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

I think it is worth thinking through if there is a possible better approach to handle adding the workspace as a "stop" in constrained move, since it has added complexity here. But if you think about it and don't have any better ideas, this LGTM.

Comment thread packages/blockly/core/dragging/block_drag_strategy.ts
Comment thread packages/blockly/core/dragging/block_drag_strategy.ts Outdated
Comment thread packages/blockly/core/dragging/block_drag_strategy.ts
@gonfunko gonfunko requested a review from maribethb April 20, 2026 18:25
@gonfunko gonfunko merged commit ca81c9a into v13 Apr 20, 2026
5 checks passed
@gonfunko gonfunko deleted the froot-loops branch April 20, 2026 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: fix Fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants