Course exhibit SVGs: per-image quality pass (refactoring.fm style) - #447
Merged
Conversation
Contributor
|
Warning Review limit reached
Next review available in: 69 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (11)
📒 Files selected for processing (23)
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 |
pftg
changed the title
… kit-sample-row (new) Paul's new exhibit bar: refactoring.fm illustration look - big bold Caveat labels with soft colored glow, saturated per-element colour-coding, tinted pills with shadow + wobble, bold blue arrows, clean white, NO emoji. Spec captured in house-visual-spec. jargon-translator = the template. kit-sample-row restored to the (better) new 4-col version per Paul's per-image verdict. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…H, rebalance, fix footer overflow Re-review caught: dropped the 'fix the architecture' lesson tie-in + DHH 'one-person framework' citation, dead space in the green card, and a footer that overflowed 720. Retitled to the jargon phrase, spaced the green items to fill, split the footer into 3 fitting lines. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
First campaign-window read (run a day early) against the week-0 baseline. No course-landing lift (84→75 sessions/14d), ~93% Direct, course absent from GSC top-25; interactive funnel dead (start_course=1, checkpoint=4, pdf/copy-share/glossary=0). Gates blog→course bridge unproven, removal of unused mechanics, and refocusing item 13 on M2-M3 depth.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Replaces the other session's flat 720 version. Structure Paul approved (3 process chips → hero '±20%?' decision → green within / amber over, no dashed-loop clutter) restyled to the locked refactoring.fm look: bold blue numbered chips + arrows, glowing hero + card headers, tinted pills with shadow + wobble, saturated green/amber/blue. Chip overflow fixed (narrower chips + shorter subs); amber trimmed to the ✓/✗. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…/right amber) Paul: two arrows hit the same block (confusing) + text too narrow. Redrawn as a proper decision fork - one green NO arrow to the left within-card, one amber YES arrow to the right over-card, side by side, concise. Matches the refactoring.fm decision-fork pattern. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…sistent steps Paul: text overflown + inconsistent. Shortened each step's body/note to fit the card (no viewBox overflow), consistent structure across all 4 steps (number + title + one body line + one italic note; step 4 = the 3 color-coded outcomes). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…presale-flow drop arrows + spacing Paul (design critique via /impeccable): stop cramming, use refactoring.fm simplicity + real visual gates. invoice-loop = one big '±20%?' question forking to two big cards (dropped the crammed process row, formula → sub, sub overflow fixed). dollar-presale-flow = removed the redundant/ill-fitted connector arrows (numbered steps already convey order) + generous even spacing. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…xes, not a text brick Paul: original presented the components well; my version was one brick of text where nobody finds the core elements. Redrawn each stack piece as its own box - green = a clean 6-box monolith stack, red = a dense 9-box grid - so the components are scannable AND 'over-engineered' is visible in the layout. DHH line kept. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…step-4 circle collision Paul (validated direction + real-page gate): invoice-loop = one continuous gauge bar split at the +20% threshold - wide green PAY, narrow amber DEMAND; the width ratio carries the meaning. dollar-presale-flow: step-4 outcomes moved clear of the number circle (were colliding). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…ngle metaphor Paul: the two box-columns looked too similar - not clear why over-engineered is worse. New metaphor makes the complexity VISIBLE: green = ONE cohesive Rails-monolith box holding all the pieces (one repo); red = a tangled web of 9 service boxes wired together (spaghetti). Simple unit vs hairball. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Mad-libs sentence with the 5 blanks as dashed fill-me-in chips (CUSTOMER, PROBLEM, APPROACH, COMPETITION, DIFFERENTIATION), plus the vague-vs-specific contrast (gray 'small businesses → a polite nod' vs ruby 'solo chiropractors → that's me!'). Basis line kept. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…ns (re-audit) Design-director re-audit: both sides were text-list bricks + the red card had a big empty gap. Reworked each item into a distinct ✓/✗ chip (green evidence chips, red banned-artifact chips), red chips taller to balance the 6-vs-4 count. Tightened the Jira chip to avoid a borderline overflow. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Old exhibit was viewBox 1000x400 muted style with ~13px labels -> ~5px @390 (mobile-illegible). Redrawn as a 2-row grid at 720 wide: 7 blue numbered question badges + an amber STOP card, amber hard-rules footer. Numbered badges carry sequence (no chevrons). All fonts >=19px, zero overflow, render-verified at 390.
36 old-style exhibits trip check-svg-floor @390 (2026-08-13). Framed per the render-gate-not-metric-gate learning: candidate list to render+eyeball, not a sweep. Opportunistic per-image migration; 5 walkthrough artifact-trails share one layout.
3 render-caught fixes: drop the card wobble (it read as misalignment in a grid, not playful), rebalance the STOP card to mirror the question rhythm (amber badge + white stop-square + short HARD STOP label, was edge-crowding '15 MIN HARD'), tighten card height 118->108 to kill the hollow badge-to-label gap. Re-rendered at 390.
Transparency artifact so the friday-demo-timeline clunkiness isn't repeated: 6-item composition checklist run after the binary gate, per-batch independent critic backstop, and the corrected wobble rule (craft in a stack, misalignment in a grid). Decision record + root cause inline.
… item 0 Manager panel (3-lens, unanimous) picked traffic/reach over course polish per the Aug-14 read. Audit correction: the LinkedIn ICP-E campaign is already built + ready (POSTING-PACKET, 10 drafts, 0 posted), blocked purely on Paul posting #1. Step 2 (UTM'd course-link arrival posts) backlogged behind step-1 engagement signal. Media polish + 36-image migration + item 7 parked/blocked.
…gn live Scheduled via claude-in-chrome, verified in LinkedIn Scheduled posts. 2-week kill-clock starts on publish today. POSTING-PACKET + tracker item 0 updated.
pftg
force-pushed
the
course-svg-per-image-fixes
branch
from
August 13, 2026 12:39
299b0de to
1e0dd6d
Compare
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.
Course exhibit SVGs — per-image quality pass (refactoring.fm style)
Rework of the course exhibit SVGs after the item18 mass-conversion was abandoned (PR #445) for flattening good hand-drawn originals. This branch is the careful per-image redo, driven by Paul's live per-image verdicts + a new visual bar.
The bar (Paul, 2026-08-13)
The refactoring.fm (Luca Rossi) illustration look — reference set in
docs/projects/2605-.../10-19-research/images/:Captured in
.okf/design/house-visual-spec.md("Refactoring.fm illustration style"). Template exhibit:jargon-translator.svg.First 10 — status
Method (the lesson from item18)
Per-image craft, not a mechanical batch: render original + candidate, keep the good of each, render-verify every result at 390 + desktop (overflow is unverifiable from source). Paul reviews per-image; nothing merges without his taste gate.
WIP — 4 done, 5 queued.
check-svg-floorstays report-only (never a blocking gate that forces redesign).🤖 Generated with Claude Code