Skip to content

feat(Flags): add Malaysia and Taiwan, standardize rounded corners#1030

Merged
DreaminDani merged 3 commits into
mainfrom
dani-flags-update
May 15, 2026
Merged

feat(Flags): add Malaysia and Taiwan, standardize rounded corners#1030
DreaminDani merged 3 commits into
mainfrom
dani-flags-update

Conversation

@DreaminDani
Copy link
Copy Markdown
Contributor

Summary

  • Adds new Malaysia and Taiwan flag assets.
  • Standardizes rounded-corner rendering on flags that were missing it (Germany, Japan, Switzerland, South-Korea, UAE, United-States) or doing it via custom path geometry (Australia, Brazil). All flags now use a clipPath rect with rx=1, matching the existing pattern.
  • Adds a dedicated Flag Storybook page (mirroring the Logo story), and moves Icon and Logo from Display/* to Assets/* so the three asset stories are grouped together.

Test plan

  • Run Storybook and visit Assets/Flag — verify the new playground renders Malaysia, Taiwan, and US flags correctly.
  • Visit Assets/Icon → "Icons" story — verify the Flags section shows all flags with consistent rounded corners (no square-cornered outliers).
  • Confirm Assets/Logo and Assets/Icon appear under the new Assets group in the sidebar (no leftover Display/* entries for these).

🤖 Generated with Claude Code

DreaminDani and others added 3 commits May 14, 2026 18:03
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Standardizes the rounded-corner approach on flags that were missing it
or doing it via custom path geometry. All flags now use a clipPath rect
with rx=1, matching the existing pattern in Mexico, Taiwan, and Malaysia.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Adds a dedicated Storybook page for Flag mirroring the Logo story, and
moves Icon and Logo from Display/* to Assets/* so the three asset
components are grouped together.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@DreaminDani DreaminDani requested review from gjones and guykoh May 15, 2026 00:04
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 15, 2026

⚠️ No Changeset found

Latest commit: 26611a1

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@workflow-authentication-public
Copy link
Copy Markdown
Contributor

Storybook Preview Deployed

✅ Preview URL: https://click-go7fryuml-clickhouse.vercel.app

Built from commit: 604e67dd59588e77ee84b260800c4586b486e02b

@DreaminDani DreaminDani enabled auto-merge (squash) May 15, 2026 00:08
@DreaminDani DreaminDani merged commit 1e9dc21 into main May 15, 2026
10 checks passed
@DreaminDani DreaminDani deleted the dani-flags-update branch May 15, 2026 02:44
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.

2 participants