Skip to content

Add gradient icon bubbles to features section - #81

Merged
thomasgallagher merged 2 commits into
mainfrom
features-gradient-icons
May 11, 2026
Merged

Add gradient icon bubbles to features section#81
thomasgallagher merged 2 commits into
mainfrom
features-gradient-icons

Conversation

@thomasgallagher

Copy link
Copy Markdown
Collaborator

Wrap each feature icon in a coloured bubble on the dummy app home page: solid black for minimal, a dark-to-light grey gradient for the theme feature, and vivid multi-stop gradients for the rest.

Wrap each feature icon in a coloured bubble on the dummy app home page:
solid black for minimal, a dark-to-light grey gradient for the theme
feature, and vivid multi-stop gradients for the rest.
@claude

claude Bot commented May 11, 2026

Copy link
Copy Markdown

Review

Wraps each feature icon in a coloured circular bubble with gradient backgrounds in the dummy app home page features partial, using two shared ERB variables to avoid repetition.

Ship - Clean, low-risk visual change with no bugs or correctness issues.

@claude

claude Bot commented May 11, 2026

Copy link
Copy Markdown

Review

Wraps each feature icon in a coloured bubble on the dummy app home page, using solid black for minimal, and vivid multi-stop gradients for the rest. A filter makes all SVG icons white against their backgrounds.

No issues found. The changes are confined to the dummy app documentation partial, WCAG decorative-image attributes are preserved, and the Tailwind gradient utilities are legitimate in this context per the dummy app policy in CLAUDE.md.

Ship

@claude

claude Bot commented May 11, 2026

Copy link
Copy Markdown

Review

Wraps each feature icon in a coloured bubble on the dummy app home page, using solid black for minimal and vivid multi-stop gradients for the rest. A CSS filter makes all SVG icons white against their backgrounds.

No issues found. The changes are confined to the dummy app documentation partial, WCAG decorative-image attributes are preserved, and the Tailwind gradient utilities are legitimate in this context per the dummy app policy in CLAUDE.md.

Ship

@thomasgallagher
thomasgallagher merged commit 463e78d into main May 11, 2026
4 checks passed
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