Skip to content

Conversation

@SupakornNetsuwan
Copy link
Collaborator

Screenshot 2568-06-04 at 15 44 27 Screenshot 2568-06-04 at 15 44 53 Screenshot 2568-06-04 at 15 45 06 Screenshot 2568-06-04 at 15 45 25 Screenshot 2568-06-04 at 15 45 34

@changeset-bot
Copy link

changeset-bot bot commented Jun 4, 2025

🦋 Changeset detected

Latest commit: d02a6d7

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

This PR includes changesets to release 0 packages

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

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

@SupakornNetsuwan SupakornNetsuwan requested review from Copilot and saenyakorn and removed request for Copilot and saenyakorn June 4, 2025 08:45
Copy link
Contributor

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 and showcases new UI components (MultipleSelect, TagGroup, Badge updates) and introduces a utility (VisuallyHidden) in the playground.

  • Introduce MultipleSelect and TagGroup components with examples in ui-playground
  • Add VisuallyHidden utility and its dependency
  • Refactor Badge spacing/size, rename CaretDown to CaretDownIcon, and mark required fields in Select

Reviewed Changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/next/src/pages/ui-playground.tsx Add playground demos for MultipleSelect, TagGroup, and Badge
packages/next/src/intentui/ui/visually-hidden.tsx New VisuallyHidden component
packages/next/src/intentui/ui/tag-group.tsx New TagGroup, TagList, and Tag components
packages/next/src/intentui/ui/multiple-select.tsx New MultipleSelect and related items
packages/next/src/intentui/ui/select.tsx Rename CaretDown to CaretDownIcon, add required indicator
packages/next/src/intentui/ui/badge.tsx Update badgeShapes spacing and badgeStyles text size
packages/next/package.json Add @react-aria/visually-hidden dependency
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (3)

packages/next/src/intentui/ui/tag-group.tsx:3

  • The hook 'use' is not valid for accessing React context. Replace it with 'useContext' and import 'useContext' from 'react'.
import { createContext, use } from 'react'

packages/next/src/intentui/ui/multiple-select.tsx:204

  • The Tailwind class 'min-w-(--trigger-width)' is invalid syntax. Use bracket notation like 'min-w-[--trigger-width]' to reference a CSS variable.
className="min-w-(--trigger-width) overflow-hidden"

packages/next/src/intentui/ui/multiple-select.tsx:211

  • Tailwind does not support 'shadow-0'; use 'shadow-none' to remove box shadow.
className="max-h-[inherit] min-w-[inherit] border-0 shadow-0"

Copy link
Member

@saenyakorn saenyakorn left a comment

Choose a reason for hiding this comment

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

ฝากเพิ่ม changeset

@SupakornNetsuwan SupakornNetsuwan merged commit 08b7558 into DRIZZ-83 Jun 7, 2025
1 check failed
@SupakornNetsuwan SupakornNetsuwan deleted the DRIZZ-93 branch June 7, 2025 16:00
This was referenced Jun 10, 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.

2 participants