docs(readme): reshoot the Overview hero with the span activity grid - #98
Closed
Fl0p wants to merge 1 commit into
Closed
docs(readme): reshoot the Overview hero with the span activity grid#98Fl0p wants to merge 1 commit into
Fl0p wants to merge 1 commit into
Conversation
The hero predates the span activity grid, so the README opened on a page that no longer exists. Shot against current main on the seeded demo instance. The crop was pinned to the second stat section, which the new grid pushed from Activity & Cost onto Users — cutting the hero short. Match the section by title instead, the way the costs shot already does, so another block landing above it does not silently reframe the shot. Co-Authored-By: Daedalus <daedalus@agents.flopbut.local> Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
Author
|
Landed on main as 8fc93a2 (squashed locally so author+committer stay Daedalus; see the merge-identity rule). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The README hero predated the span activity grid (#97), so the page it advertised no longer exists. Re-shot against current
mainon the seeded throwaway demo instance, perdocs/operations/screenshots.md. One shoot covers both of yesterday's Overview changes (#96, #97).The crop needed a fix to survive it.
dashboard-overviewwas cut at the second stat section; the grid landing above pushed that index from Activity & Cost onto Users, which would have chopped the hero short. It now matches the section by title — the same shapedashboard-costsalready uses — so a future block above it does not silently reframe the shot.Hero now reads: KPI row → Span activity grid → Users → Activity & Cost.
Only
dashboard-overview.pngis committed; the other four pages are unchanged in shape and their re-shoots were reverted rather than commit shoot-time drift.🤖 Generated with Claude Code