Skip to content

chore(website): refresh homepage inspector screenshot and remove dark toggle#5360

Merged
NicholasKissel merged 1 commit into
mainfrom
NicholasKissel/update-inspector-screenshot
Jul 5, 2026
Merged

chore(website): refresh homepage inspector screenshot and remove dark toggle#5360
NicholasKissel merged 1 commit into
mainfrom
NicholasKissel/update-inspector-screenshot

Conversation

@NicholasKissel

Copy link
Copy Markdown
Member
  • swap the observability inspector screenshot to the new light-only asset
  • remove the light/dark toggle and dark variant logic
  • update the screenshot frame aspect ratio to match the new image

@railway-app railway-app Bot temporarily deployed to rivet-frontend / rivet-pr-5360 July 3, 2026 01:30 Destroyed
@railway-app

railway-app Bot commented Jul 3, 2026

Copy link
Copy Markdown

🚅 Deployed to the rivet-pr-5360 environment in rivet-frontend

Service Status Web Updated (UTC)
frontend-cloud 😴 Sleeping (View Logs) Web Jul 5, 2026 at 8:36 pm
website 😴 Sleeping (View Logs) Web Jul 5, 2026 at 3:49 am
frontend-inspector 😴 Sleeping (View Logs) Web Jul 4, 2026 at 8:15 pm
kitchen-sink 😴 Sleeping (View Logs) Web Jul 4, 2026 at 7:12 pm
mcp-hub ✅ Success (View Logs) Web Jul 4, 2026 at 6:52 pm
ladle ✅ Success (View Logs) Web Jul 4, 2026 at 6:52 pm

@NicholasKissel NicholasKissel force-pushed the NicholasKissel/update-inspector-screenshot branch from c5664b5 to 1ed09c7 Compare July 4, 2026 18:51
@railway-app railway-app Bot temporarily deployed to rivet-frontend / rivet-pr-5360 July 4, 2026 18:51 Destroyed
@NicholasKissel NicholasKissel reopened this Jul 4, 2026
@railway-app railway-app Bot temporarily deployed to rivet-frontend / rivet-pr-5360 July 4, 2026 18:52 Destroyed
@claude

claude Bot commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Review

Small, well-scoped cleanup: swaps the inspector screenshot for the new light-only asset and removes the now-dead light/dark toggle state, images map, and unused imports (Sun, Moon, useState).

Checked, no issues found:

  • No leftover references to the removed inspectorImages/InspectorTheme/toggle code, and motion is still used elsewhere in the file (for motion.div/motion.p), so no unused imports remain.
  • ObservabilitySection isn't consumed anywhere else in a way that depended on the removed exports (only actors.astro / index.astro render <ObservabilitySection /> directly).
  • No CLAUDE.md conventions (website typography/theme rules) are violated by this change.

One thing worth a quick manual check before merge: inspectorAspect was changed from 2438 / 1613 to 2688 / 2018. Since this drives the aspectRatio CSS on the screenshot frame, worth confirming that 2688 / 2018 actually matches the real pixel dimensions of the new rivet-actor-inspector.png asset — a mismatch here wouldn't break the build but would show the screenshot slightly stretched/cropped in the frame. I wasn't able to fetch the asset directly to verify dimensions in this environment.

Otherwise this looks good to merge.

@NicholasKissel NicholasKissel merged commit 88a378b into main Jul 5, 2026
12 of 16 checks passed
@NicholasKissel NicholasKissel deleted the NicholasKissel/update-inspector-screenshot branch July 5, 2026 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant