Ensure black on white for move indicator. (#612)

microbit-matt-hillsdon · web-flow · commit 8e2b18e85dbb · 2025-06-23T08:51:47.000-07:00
This fixes it in MakeCode's dark theme that sets color to white at a high level in the DOM which is then used on semi-transparent white here via currentColor.
Seems reasonable to hardcoded the foreground if we do so for the background.
diff --git a/src/move_indicator.ts b/src/move_indicator.ts
@@ -53,7 +53,7 @@ export class MoveIndicatorBubble
       Blockly.utils.Svg.PATH,
       {
         'fill': 'none',
-        'stroke': 'currentColor',
+        'stroke': 'black',
         'stroke-linecap': 'round',
         'stroke-linejoin': 'round',
         'stroke-width': '2',

Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ export class MoveIndicatorBubble`
`53`	`53`	`Blockly.utils.Svg.PATH,`
`54`	`54`	`{`
`55`	`55`	`'fill': 'none',`
`56`		`- 'stroke': 'currentColor',`
	`56`	`+ 'stroke': 'black',`
`57`	`57`	`'stroke-linecap': 'round',`
`58`	`58`	`'stroke-linejoin': 'round',`
`59`	`59`	`'stroke-width': '2',`