Amber Console restyle of 3d components (follow-up to #154) - #159
Conversation
The design pass over CellDetail, MembersTooltip, VoxelView, and Scatter3D was applied in the working tree during the feature/3d merge but never staged, so #154 landed without it. This completes it: CellDetail drawer on the PointDetail drawer treatment (ls-icon-btn close, shared Pagination, mono machine facts), MembersTooltip on floating-panel tokens instead of the inverse-copper card, VoxelView and Scatter3D reading their canvas clear colors from --ls-surface-map via useColorMode. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ef89b05949
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @@ -1,5 +1,7 @@ | |||
| import { createPortal } from 'react-dom'; | |||
|
|
|||
| import styles from './MembersTooltip.module.scss'; | |||
There was a problem hiding this comment.
Importing this CSS module breaks the web bundle because web/src/components/Explore/MembersTooltip.module.scss is not present in this commit (I searched the repo files and only found MembersTooltip.jsx). Any Explore path that imports MembersTooltip will fail module resolution in Vite before the UI can load, so either add the stylesheet or keep the styles in an existing file.
Useful? React with 👍 / 👎.
1 participant
The restyle of CellDetail, MembersTooltip, VoxelView, and Scatter3D was in the working tree during the #154 merge but never staged, so it missed the merge commit. This is that work: token-driven drawer/tooltip/HUD chrome and theme-reactive canvas clear colors, with data-viz palettes untouched. Build, eslint, and vitest verified.
🤖 Generated with Claude Code