Skip to content

Conversation

avivkeller
Copy link
Member

@avivkeller avivkeller commented May 31, 2025

This is the CircularIcon component, based on the design shown below:

Design reference

Storybook preview:

Storybook preview

@Copilot Copilot AI review requested due to automatic review settings May 31, 2025 17:26
@avivkeller avivkeller requested a review from a team as a code owner May 31, 2025 17:26
Copy link

vercel bot commented May 31, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nodejs-org ✅ Ready (Inspect) Visit Preview Jun 1, 2025 1:17pm

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces a new UI component called CircularIcon that displays a circular icon with customizable symbol, color, and size. Key changes include:

  • Adding the CircularIcon component in TypeScript.
  • Including a Storybook story to demonstrate various icon sizes.
  • Adding a CSS module for styling the component with Tailwind CSS utilities.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
packages/ui-components/Common/CircularIcon/index.tsx Adds the CircularIcon component implementation.
packages/ui-components/Common/CircularIcon/index.stories.tsx Provides Storybook stories for the CircularIcon.
packages/ui-components/Common/CircularIcon/index.module.css Implements the styling for the CircularIcon component using CSS modules.

Copy link

codecov bot commented May 31, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.46%. Comparing base (3ecfbcb) to head (7e2faf1).
Report is 10 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7797      +/-   ##
==========================================
+ Coverage   75.34%   75.46%   +0.11%     
==========================================
  Files          96      101       +5     
  Lines        7862     8309     +447     
  Branches      192      218      +26     
==========================================
+ Hits         5924     6270     +346     
- Misses       1937     2037     +100     
- Partials        1        2       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Co-authored-by: Caner Akdas <[email protected]>
Signed-off-by: Aviv Keller <[email protected]>
Copy link
Contributor

github-actions bot commented May 31, 2025

Lighthouse Results

URL Performance Accessibility Best Practices SEO Report
/en 🟢 99 🟢 100 🟢 100 🟢 91 🔗
/en/about 🟢 100 🟢 100 🟢 100 🟠 82 🔗
/en/about/previous-releases 🟢 100 🟢 100 🟢 100 🟠 83 🔗
/en/download 🟢 97 🟢 100 🟢 100 🟢 91 🔗
/en/blog 🟢 100 🟢 100 🟢 96 🟢 92 🔗

@avivkeller avivkeller changed the title feat(ui): add CircularIcon feat(ui): add DataTag May 31, 2025
Copy link
Member

@AugustinMauroy AugustinMauroy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ! That's clean

@avivkeller avivkeller added fast-track Fast Tracking PRs github_actions:pull-request Trigger Pull Request Checks labels Jun 1, 2025
@github-actions github-actions bot removed the github_actions:pull-request Trigger Pull Request Checks label Jun 1, 2025
@avivkeller
Copy link
Member Author

Fast-tracking so I can add this to the web generator. If you approve, merge when ready.

@ovflowd ovflowd added this pull request to the merge queue Jun 1, 2025
Merged via the queue into main with commit 79dc769 Jun 1, 2025
26 checks passed
@ovflowd ovflowd deleted the feat/CircularIcon branch June 1, 2025 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast-track Fast Tracking PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants