feat(web): align office wall fixtures and expand windows/plants#4
feat(web): align office wall fixtures and expand windows/plants#4charlesrhoward merged 1 commit intomainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![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: 6a13ed896b
ℹ️ 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".
| {/* Monitor wall */} | ||
| <MonitorWall position={[10.8, 0, -3]} /> | ||
| {/* Monitor wall mounted flush on right wall */} | ||
| <MonitorWall position={[10.88, 0, -5]} rotation={[0, -Math.PI / 2, 0]} /> |
There was a problem hiding this comment.
The new MonitorWall placement at position={[10.88, 0, -5]} puts the monitors directly into the same wall section as the right window centered at z=-5.4 (right-window instances), so part of the top row overlaps the window frame/glass volume. In practice this creates intersecting geometry (visible clipping/flicker) and makes the wall fixtures visually inconsistent; placing the monitor wall in a non-window span of the right wall would avoid the collision.
Useful? React with 👍 / 👎.
1 participant
Note
Low Risk
Purely visual/layout changes to a single canvas component; no data flow, auth, or backend logic impacted.
Overview
Adjusts the 3D
Officescene so the monitor wall can be rotated and is repositioned to sit flush on the right wall.MonitorWallnow accepts an optionalrotationprop (defaulting to no rotation), and theOfficeplacement updates the monitor wall position and rotates it by 90° to align with the right-side wall plane.Written by Cursor Bugbot for commit 6a13ed8. This will update automatically on new commits. Configure here.