Skip to content

refactor: deduplicate FAQAnswer blue/white rendering#279

Merged
andygrunwald merged 1 commit intomainfrom
refactor/faq-answer-dedup
Mar 21, 2026
Merged

refactor: deduplicate FAQAnswer blue/white rendering#279
andygrunwald merged 1 commit intomainfrom
refactor/faq-answer-dedup

Conversation

@andygrunwald
Copy link
Member

Summary

  • Replace two ~85% identical template branches with a single template using conditional CSS classes
  • Only background colors, tag colors, title colors, and text colors differ between variants
  • Reduces component from 43 lines to 32 lines

Test plan

  • astro build succeeds
  • Verify FAQ page shows correct alternating blue/white card styling

🤖 Generated with Claude Code

Replace two ~85% identical template branches with a single template
using conditional CSS classes for background, tag, title, and text
colors based on the blue flag.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@netlify
Copy link

netlify bot commented Mar 21, 2026

Deploy Preview for lively-frangollo-e43a02 ready!

Name Link
🔨 Latest commit c24ec3a
🔍 Latest deploy log https://app.netlify.com/projects/lively-frangollo-e43a02/deploys/69bf00ac6e18330008a4c2a6
😎 Deploy Preview https://deploy-preview-279--lively-frangollo-e43a02.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@andygrunwald andygrunwald merged commit 29ace1f into main Mar 21, 2026
5 checks passed
@andygrunwald andygrunwald deleted the refactor/faq-answer-dedup branch March 21, 2026 20:45
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