Skip to content

fix: transform svgPathSelected when using RTL and Zelos#10152

Merged
mikeharv merged 2 commits into
RaspberryPiFoundation:mainfrom
mikeharv:solez
Jul 17, 2026
Merged

fix: transform svgPathSelected when using RTL and Zelos#10152
mikeharv merged 2 commits into
RaspberryPiFoundation:mainfrom
mikeharv:solez

Conversation

@mikeharv

Copy link
Copy Markdown
Contributor

The basics

The details

Resolves

Fixes #10151

Proposed Changes

Update flipRTL() to also transform svgPathSelected when it exists. This is the same pattern Geras uses for its light/dark paths in geras/path_object.ts.

Reason for Changes

From the issue:

With Zelos in RTL mode, when one drops a block from the toolbox, the yellow border is mirrored in LTR.

image

Test Coverage

Documentation

Additional Information

@mikeharv
mikeharv requested a review from a team as a code owner July 17, 2026 13:14
@mikeharv
mikeharv requested a review from gonfunko July 17, 2026 13:14
@mikeharv mikeharv changed the title Solez fix: transform svgPathSelected when using RTL and Zelos. Jul 17, 2026
@github-actions github-actions Bot added the PR: fix Fixes a bug label Jul 17, 2026
@mikeharv mikeharv changed the title fix: transform svgPathSelected when using RTL and Zelos. fix: transform svgPathSelected when using RTL and Zelos Jul 17, 2026
@github-actions github-actions Bot added PR: fix Fixes a bug and removed PR: fix Fixes a bug labels Jul 17, 2026

@gonfunko gonfunko left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This still works on the main workspace too right? I know I made some change to fix this bug there but must have missed this case.

@mikeharv

Copy link
Copy Markdown
Contributor Author

This still works on the main workspace too right? I know I made some change to fix this bug there but must have missed this case.

Yes, I'm not seeing any regressions.

@mikeharv
mikeharv merged commit 788399c into RaspberryPiFoundation:main Jul 17, 2026
13 of 17 checks passed
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.

Glitch when dropping a block from toolbox with Zelos in RTL mode in v13

2 participants