diff --git a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Client-side-scene-scale-should-match-engine-scale-Inch-scale-1-Google-Chrome-linux.png b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Client-side-scene-scale-should-match-engine-scale-Inch-scale-1-Google-Chrome-linux.png index e51af3536..623762542 100644 Binary files a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Client-side-scene-scale-should-match-engine-scale-Inch-scale-1-Google-Chrome-linux.png and b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Client-side-scene-scale-should-match-engine-scale-Inch-scale-1-Google-Chrome-linux.png differ diff --git a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Client-side-scene-scale-should-match-engine-scale-Millimeter-scale-1-Google-Chrome-linux.png b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Client-side-scene-scale-should-match-engine-scale-Millimeter-scale-1-Google-Chrome-linux.png index 1426727c1..d45186bf4 100644 Binary files a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Client-side-scene-scale-should-match-engine-scale-Millimeter-scale-1-Google-Chrome-linux.png and b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Client-side-scene-scale-should-match-engine-scale-Millimeter-scale-1-Google-Chrome-linux.png differ diff --git a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-segments-should-look-right-2-Google-Chrome-linux.png b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-segments-should-look-right-2-Google-Chrome-linux.png index 6742af07c..cef99a896 100644 Binary files a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-segments-should-look-right-2-Google-Chrome-linux.png and b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-segments-should-look-right-2-Google-Chrome-linux.png differ diff --git a/src/Toolbar.tsx b/src/Toolbar.tsx index 03a00b2f7..0ac75eba5 100644 --- a/src/Toolbar.tsx +++ b/src/Toolbar.tsx @@ -190,49 +190,59 @@ export function Toolbar({ maybeIconConfig[0].onClick(configCallbackProps) } > - + + {maybeIconConfig[0].title} + + ) } const itemConfig = maybeIconConfig return ( - itemConfig.onClick(configCallbackProps)} - > - + itemConfig.onClick(configCallbackProps)} + > + + {itemConfig.title} + + + - + ) })} @@ -250,7 +260,7 @@ export function Toolbar({ * It contains a tooltip with the title, description, and links * and a hotkey listener */ -const ToolbarItemContents = memo(function ToolbarItemContents({ +const ToolbarItemTooltip = memo(function ToolbarItemContents({ itemConfig, configCallbackProps, }: { @@ -272,73 +282,69 @@ const ToolbarItemContents = memo(function ToolbarItemContents({ ) return ( - <> - - {itemConfig.title} - - -
- - {itemConfig.title} - - {itemConfig.status === 'available' && itemConfig.hotkey ? ( - {itemConfig.hotkey} - ) : itemConfig.status === 'kcl-only' ? ( + +
+ + {itemConfig.title} + + {itemConfig.status === 'available' && itemConfig.hotkey ? ( + {itemConfig.hotkey} + ) : itemConfig.status === 'kcl-only' ? ( + <> + + KCL code only + + + + ) : ( + itemConfig.status === 'unavailable' && ( <> - KCL code only + In development - ) : ( - itemConfig.status === 'unavailable' && ( - <> - - In development - - - - ) - )} -
-

{itemConfig.description}

- {itemConfig.links.length > 0 && ( - <> -
- - + ) )} -
- +
+

{itemConfig.description}

+ {itemConfig.links.length > 0 && ( + <> +
+ + + )} +
) }) diff --git a/src/components/Tooltip.module.css b/src/components/Tooltip.module.css index a1fd8dcaf..033f2c7a5 100644 --- a/src/components/Tooltip.module.css +++ b/src/components/Tooltip.module.css @@ -57,7 +57,8 @@ transition-delay: var(--_delay); } -:is(:focus-visible) > .tooltipWrapper.withFocus { +:is(:focus-visible) > .tooltipWrapper.withFocus, +:focus-within > .tooltipWrapper.withFocus { visibility: visible; opacity: 1; } diff --git a/src/components/Tooltip.tsx b/src/components/Tooltip.tsx index 58c4d3f3b..8e0167c08 100644 --- a/src/components/Tooltip.tsx +++ b/src/components/Tooltip.tsx @@ -29,7 +29,7 @@ export default function Tooltip({ return (