Build against chobble-template main; compact, corporate theme refresh#7
Merged
Conversation
Template / build: - Remove the pinned chobble-template git submodule and build against chobble-template main from GitHub (the build scripts already clone it). - Migrate page content to the current template schema: drop the renamed design-system-base.html layout so pages use the default base.html, and convert hero blocks from name/lead to the content field. - Exclude VOICE.md from the published site (now that src has a default layout). - Drop the @chobble/js-toolkit dependency (only sourced from the removed submodule) and reimplement precommit without it. Design: - Compact vertical rhythm by pulling --space-3xl in, which feeds every section-padding variable (~12% shorter homepage). - Sharper corners and a crisp white nav bar with a brand-red top accent; tighten the menu so all items hold a single line instead of wrapping. - Enlarge the nav logo and add a tightly-trimmed brand mark so it fills the bar instead of floating in whitespace. - Add brand-red uppercase "eyebrow" labels above split headings and a thin brand rule atop feature cards for a more deliberate, corporate feel. https://claude.ai/code/session_01SGhcb3D7nyGY8P55V7zJsb
The hover lift used a broad `.design-system article:hover`, which also matched the <article class="stack"> wrapper around the prose on every split block — so hovering body text lifted and shadowed the whole text column. Scope the lift to .feature cards, where it was intended. https://claude.ai/code/session_01SGhcb3D7nyGY8P55V7zJsb
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
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.
Summary
Makes the design more compact and professional, and removes the stale pinned
chobble-templatesubmodule so the site builds againstchobble-templatemain from GitHub.While wiring up the build, the content turned out to be incompatible with both the pinned submodule (too old) and
main(renamed layout, changedheroschema). This PR migrates the content to the currentmainschema so the site builds, then applies the visual refresh.Build / template changes
chobble-templategit submodule and build againstchobble-templatemain from GitHub (the build scripts already clone it; the submodule was only providing the@chobble/js-toolkitdev dependency).design-system-base.htmllayout so pages use main's defaultbase.html, and convert theheroblocks fromname/leadto the currentcontentfield (not-found.md,thank-you.md).VOICE.mdfrom the published site — main now applies a default layout to everything undersrc/, so the docs file was being rendered as a page.@chobble/js-toolkitdependency (only sourced from the removed submodule) and reimplementscripts/precommit.jsto orchestrate lint/cpd/build without it.Design changes (
css/theme.scss)--space-3xlin, which feeds every--section-padding-y-*variable — roughly 12% shorter homepage while staying readable.--border-radius6px → 4px) for a more corporate feel.brand-mark.png(the source PNG had heavy baked-in whitespace, so it floated small in the nav).Verification
Built cleanly against
chobble-templatemain and screenshotted at mobile/tablet/desktop. Nav now sits on one line with a filled logo; migratedheropages (thank-you / 404) render correctly; content pages unaffected.https://claude.ai/code/session_01SGhcb3D7nyGY8P55V7zJsb
Generated by Claude Code