Skip to content

fix: Add class to shadow blocks in Geras#9564

Merged
gonfunko merged 2 commits intomainfrom
blockly-shadow
Jan 20, 2026
Merged

fix: Add class to shadow blocks in Geras#9564
gonfunko merged 2 commits intomainfrom
blockly-shadow

Conversation

@gonfunko
Copy link
Contributor

The basics

The details

Resolves

Fixes #9465

Proposed Changes

This PR updates Geras' path object to call super in updateShadow_, which adds the blocklyShadow class, allowing it to be reliably used in stylesheets for styling shadow blocks. It also removes checks for style.colourTertiary and style.colourSecondary; at the time these were added, those fields were nullable, but they no longer are and the presence of the fields is enforced/backfilled by Constants.validatedBlockStyle_().

@gonfunko gonfunko requested a review from a team as a code owner January 12, 2026 20:48
@gonfunko gonfunko requested a review from maribethb January 12, 2026 20:48
@github-actions github-actions bot added the PR: fix Fixes a bug label Jan 12, 2026
@gonfunko gonfunko added this to Blockly Jan 12, 2026
@github-project-automation github-project-automation bot moved this to Todo in Blockly Jan 12, 2026
@gonfunko gonfunko moved this from Todo to In Progress in Blockly Jan 12, 2026
@gonfunko gonfunko merged commit 8e9b95f into main Jan 20, 2026
12 checks passed
@gonfunko gonfunko deleted the blockly-shadow branch January 20, 2026 16:14
@github-project-automation github-project-automation bot moved this from In Progress to Done in Blockly 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

Status: Done

Development

Successfully merging this pull request may close these issues.

blocklyShadow not set when using geras

2 participants

Comments