Skip to content

Conversation

@gonfunko
Copy link
Contributor

The basics

The details

Resolves

Proposed Changes

This PR is a bit of a grab bag of flyout related fixes. Specifically, it:

  • Removes a bunch of special handling of variable and block creation events when dragging a block out of the flyout. Experimentally this still seems to work fine, including undo, and including blocks that reference potential variables, and all tests still pass.
  • Avoids a race condition that could result in inadvertent block bumps when dragging out a new block by calling appendInternal() instead of append(), the latter of which triggers an immediate (and unnecessary) render before the state of things is fully combobulated.
  • Changes the behavior such that dragging on a block will not ever drag-to-scroll the flyout. The flyout can still be scrolled by clicking and dragging on the flyout workspace background, but clicks targeting blocks will always be treated as a drag on the block.

@gonfunko gonfunko requested a review from maribethb January 20, 2026 23:22
@gonfunko gonfunko requested a review from a team as a code owner January 20, 2026 23:22
@github-actions github-actions bot added the PR: fix Fixes a bug label Jan 20, 2026
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

3 participants