Skip to content

fix(modal): consistent line-art icons for AWS Blocks feature points#8609

Closed
harsh62 wants to merge 1 commit into
mainfrom
docs/fix-blocks-modal-icons
Closed

fix(modal): consistent line-art icons for AWS Blocks feature points#8609
harsh62 wants to merge 1 commit into
mainfrom
docs/fix-blocks-modal-icons

Conversation

@harsh62

@harsh62 harsh62 commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

The AWS Blocks announcement modal (merged in #8606) had two icon issues:

  • The first feature point used a puzzle emoji, which clashes with the site's monochrome line-art icon style and renders differently across OSes.
  • The AWS icon on the second point had no explicit size, so it rendered too small and looked misaligned next to its heading.

Fix: replace the emoji with IconStar and size both feature-point icon slots identically (textAlign="center", fontSize="xxl") so the two icons match and align with their headings.

Verified with a rendered screenshot; Modal unit tests pass (5/5).

Replace the puzzle emoji with IconStar and size both feature-point icon
slots identically (textAlign center, fontSize xxl) so IconStar and
IconAWS render at a matching size and align with their headings. The
emoji clashed with the site monochrome icon style and the AWS icon was
unsized (rendered too small).
@harsh62 harsh62 requested a review from a team as a code owner July 3, 2026 15:43
harsh62 added a commit that referenced this pull request Jul 3, 2026
…l icons

Adds an "AWS Blocks integration" section to the steering-files page with
a per-tool recipe (Claude Code, Cursor, Kiro) so AI coding assistants
follow the Blocks integration patterns. All claims verified against the
aws-blocks source (create-blocks-app scaffolds AGENTS.md; backend in
aws-blocks/index.ts with CognitoVerifier + requireAuth; blocks:generate-client;
import { api } from aws-blocks; npm run sandbox).

Also folds in the modal icon fix from #8609: replace the puzzle emoji
with IconStar and size both feature-point icons consistently.

Rebuilt on latest main so the aws-blocks internal links (merged in #8606)
resolve, clearing the earlier CheckPRLinks failure.
@harsh62

harsh62 commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

Folded into #8608 (commit 30f4e08) so the steering-file docs and this modal icon fix are reviewed together. The AWS Blocks pages the steering section links to are now merged on main, so #8608's link check passes. Closing this in favor of #8608.

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