Skip to content

Conversation

tenphi
Copy link
Member

@tenphi tenphi commented Aug 19, 2025

No description provided.

Copy link

changeset-bot bot commented Aug 19, 2025

🦋 Changeset detected

Latest commit: 2e8fe83

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@cube-dev/ui-kit Patch

Not sure what this means? Click here to learn what changesets are.

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

@tenphi tenphi requested a review from Copilot August 19, 2025 09:38
Copy link

vercel bot commented Aug 19, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
cube-ui-kit Ready Ready Preview Comment Aug 19, 2025 9:52am
cube-ui-kit-cursor Ready Ready Preview Comment Aug 19, 2025 9:52am

Copy link
Contributor

github-actions bot commented Aug 19, 2025

📦 NPM canary release

Deployed canary version 0.0.0-canary-dd5a11a.

Copy link
Contributor

github-actions bot commented Aug 19, 2025

🏋️ Size limit report

Name Size Passed?
All 278 KB (+0.03% 🔺) Yes 🎉
Tree shaking (just a Button) 23.6 KB (0% 🟰) Yes 🎉
Tree shaking (just an Icon) 12.14 KB (0% 🟰) Yes 🎉

Click here if you want to find out what is changed in this build

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 adds loading state functionality to the ItemBase component and ItemButton, while also updating Storybook imports across the codebase. The loading feature allows components to display a loading icon in various slots while automatically disabling interaction.

  • Added isLoading and loadingSlot props to ItemBase for flexible loading state management
  • Updated ItemButton to use the new htmlType prop instead of the deprecated buttonType
  • Migrated all Storybook imports from framework-specific packages to the unified @storybook/react and @storybook/test packages

Reviewed Changes

Copilot reviewed 62 out of 63 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/components/content/ItemBase/ItemBase.tsx Added loading state props and logic with auto slot selection
src/components/actions/ItemButton/ItemButton.tsx Updated to use htmlType prop and support loading state
src/components/fields/Select/Select.tsx Updated button prop from buttonType to htmlType
Multiple .stories.tsx files Migrated Storybook imports to unified packages
package.json Added @storybook/react and @storybook/test dependencies
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

Copy link
Contributor

github-actions bot commented Aug 19, 2025

🧪 Storybook is successfully deployed!

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@tenphi tenphi merged commit 2400dd2 into main Aug 19, 2025
14 checks passed
@tenphi tenphi deleted the feat-item-loading-state branch August 19, 2025 10:02
@github-actions github-actions bot mentioned this pull request Aug 19, 2025
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