fix: Focus the first element in flyouts. by gonfunko · Pull Request #9228 · RaspberryPiFoundation/blockly
Navigation Menu
{{ message }}
RaspberryPiFoundation / blockly Public
- Notifications You must be signed in to change notification settings
- Fork 3.8k
Merged
Conversation
Copy link Copy Markdown
Contributor
gonfunko
commented
Jul 10, 2025
gonfunko
commented
The basics
The details
Resolves
Fixes #9103
Proposed Changes
This PR returns the first focusable node of a flyout workspace when getRestoredFocusableNode() is called, regardless of whether that element is a block or some other type of entity.
gonfunko
requested a review
from a team
as a code owner
gonfunko
requested a review
from maribethb
github-actions
bot
assigned
maribethb
maribethb approved these changes Jul 11, 2025
core/workspace_svg.ts
Show resolved
Hide resolved
core/workspace_svg.ts Show resolved Hide resolved
core/workspace_svg.ts
Outdated
Show resolved
Hide resolved
core/workspace_svg.ts Outdated Show resolved Hide resolved
core/workspace_svg.ts
Show resolved
Hide resolved
core/workspace_svg.ts Show resolved Hide resolved
gonfunko
deleted the
flyout-focus
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment