Repository navigation
design: solutions-page - Add hero, route tiles and comparison - #99
Conversation
Patterns
- Add Hero - Solutions with a tilted palette specimen card
- Add Card - Route Tile as a reusable single-card pattern
- Add Section - Solutions Route Tiles (7 cards, 3-column grid)
- Add Section - Split Intro Pair Cards for the solutions vs services comparison
Styling
- Add solutions-hero.scss for the specimen card tilt and swatch grid
- Compile assets/css/solutions-hero.css
- Reuse Card - Service Tile style and corner-glow, no new card style
Wiring
- Register solutions-hero bundle in package.json build scripts
- Load solutions-hero.css on the solutions page in inc/animations.php
- Add solutions-hero.css to editor styles in functions.php
- Widen the service tiles bundle condition to services and solutions
Content
- Source copy and links from the Solutions Landing content doc
- Link cards to /solutions/{wordpress,woocommerce,publishing,tour-operators,ai,design-systems,ai-chatbots}/
- Point primary CTA to /free-consultation/
Eyebrows - Set eyebrow font size to token 100 in Hero - Solutions - Set eyebrow font size to token 100 in Section - Solutions Route Tiles - Set eyebrow font size to token 100 in Section - Split Intro Pair Cards - Now matches the Services patterns Content - Restore Hero - Solutions eyebrow to "Solutions"
Spacing - Set card blockGap to spacing 10 on Card - Route Tile - Set card blockGap to spacing 10 on Section - Solutions Route Tiles - Remove extra spacing 20 top margin from the tile content wrapper - All vertical gaps inside the card now use the same token Typography - Raise tile title (H3) from font size 300 to 400 - Title now reads larger than the 200 kicker line
Block gaps - Hero: set outer wrapper and text column gap to spacing 20 - Routes section: set wrapper gap to spacing 60 between heading row and cards - Routes section: set heading column gap to spacing 10 - Pair cards: set left column gap to spacing 10, right column gap to spacing 30 - Remove all margin top attributes used for spacing Cleanup - Drop redundant margin-top auto from the card CTA - CTA stays bottom-anchored through the card content layout
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (12)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughAdds WordPress patterns for a Solutions hero, seven route tiles, a reusable route-tile card, and a Services/Solutions comparison. Adds scoped styles and registers their build, editor, and page-based loading; service-tile styles also load on the Solutions page. ChangesSolutions landing page
Priority: ⬇️ Low Estimated code review effort: 3 (Moderate) | ~20 minutes Change: Feature Suggested reviewers: Merge Risk: ⚪ Minimal · up to No actionable merge-blocking issue is established for the Solutions patterns and styles; the change is mergeable after normal checks. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 7 systems. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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 |
Changelog - Add Unreleased entry for the Solutions hero, route tiles, route card and comparison section - Link the entry to PR #99 Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…olutions-page-stats-and-cta Stack - Bring the first layer's CHANGELOG entry for PR #99 into this layer Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Design Summary
The Solutions landing page (
/solutions/) needs its first three sections built as reusablels-themepatterns: a hero, a "solution selector" grid of seven route cards, and a short solutions-versus-services comparison. This adds those patterns, a reusable single route card, and the small amount of SCSS and wiring they need.Patterns
patterns/hero/solutions-hero.php: same skeleton as the Services hero (breadcrumb, eyebrow, H1, intro, two CTAs) plus a tilted "solutions / palette" specimen card. The background reuses the sharedcorner-glowclass. Primary CTA goes to/free-consultation/, secondary to/services/.patterns/sections/solutions-route-tiles.php: eyebrow, heading and intro above a grid of seven route tiles (WordPress, WooCommerce, Publishing, Tour Operators, AI, Design Systems, AI Chatbots). Each tile is one stretched link to its/solutions/{slug}/page.patterns/cards/card-route-tile.php: the same tile as a single editable card pattern. Both patterns use the existing Card - Service Tile style, so no newis-styleentry was added. The two keep the same markup, which is noted in each description.patterns/sections/split-intro-pair-cards.php: eyebrow and heading beside a paragraph and two bordered cards comparing services and solutions.margin-top: 0reset, which matches the Services hero.Styling (
src/scss/structural/, compiled toassets/css/)solutions-hero.scss: the stacked, tilted specimen card and its swatch grid. Kept in SCSS because theme.json has no key for position, transform or an aspect-ratio grid. Each rule carries a// JSON limitation:comment.solutions-route-tiles.scss: scoped to.ls-route-tile, so the Services tiles and Work archive icon wells are unchanged. It sets a larger icon well and icon from spacing tokens and insets the index number to match the card padding.Wiring
package.json: both new SCSS files added tobuild:css,build:css:devandwatch:css.inc/animations.php: both bundles load on the Solutions page viais_page( 'solutions' ), with render-marker fallbacks; the existing service tiles bundle now loads onservicesandsolutions.functions.php: both stylesheets added as editor styles.Content
/solutions/{slug}/paths. The doc's "Publishers" is the real/solutions/publishing/page.Linked Stories/Tasks/PRs/Design Files / Related Epics
Relates to Linear LSA-128
Milestones & Timeline
No milestone, release date or deadline supplied. Stack sequence: #99 first, followed by #100.
Designs / Mockups
Accessibility Considerations
100, matching the existing Services eyebrows.Acceptance Criteria
Review checklist; completion has not been independently verified.
The existing branch is
feat/lsa-128-solutions-page-hero-and-routes; it has not been renamed. The template'sdesign/criterion remains unverified.Dependencies
develop)Testing/Validation Steps
Existing checked results below are author-reported evidence, not new validations performed during this description update.
php -lon all 6 changed PHP filesphpcs(WordPress standard) on all 6 changed PHP files: no errors or warningsnpm run schema:validateandnpm run lint:jsonnpm run security:scannpm run patterns:escape: none of the changed files flagged (26 existing warnings elsewhere)npm run build:css: no diff after rebuildingnpm run theme:validate: completes with warnings only, all from.github/promptsplaceholders unrelated to this changehttp://localhost:8882/solutions/in light and dark mode (chromium, webkit, Mobile Chrome accessibility): 27 passed in each mode (the local page also contained the next layer's two patterns during these runs)develop: 0 findingsAdditional Context
Scope and exclusions
assets/css/that is 9 reviewable files and about +572 lines, over the preferred ~400 lines but well inside the 800-line limit. The patterns are mostly block markup.Notes for reviewers
destinationicon because the icon set has no map icon.border.card-subtletoken. It is proposed but not added here, so the card border is unchanged.theme_filestransient) if new patterns do not show after switching branches locally.Changelog
meta:needs-changelog: user-facing new patterns. The original description states that the entry is added toCHANGELOG.mdafter the PR is created and links to it; this update does not independently verify that entry.🤖 Generated with Claude Code
Summary by CodeRabbit
Definition of Ready (DoR)
Definition of Done (DoD)