Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Dropdown 组件新增 shape: 同 Button 组件的shape #925

Open
wants to merge 3 commits into
base: dev-3.6.0
Choose a base branch
from

Conversation

saint3347
Copy link
Collaborator

Types of changes

  • New feature
  • Bug fix
  • Site / documentation update
  • Demo update
  • Component style update
  • TypeScript definition update
  • Bundle size optimization
  • Performance optimization
  • Enhancement feature
  • Internationalization
  • Refactoring
  • Code style optimization
  • Test Case
  • Branch merge
  • Workflow
  • Others

Background

Information Descriptions
Browser Chrome / Safari /
Version Chrome 49 ...
OS MacOS / Windows ...

Changelog

  • Dropdown 组件新增 buttonShape: 同 Button 组件的shape

Playground id

Other information

@saint3347 saint3347 changed the title feat: Dropdown 组件新增 buttonShape: 同 Button 组件的shape feat: Dropdown 组件新增 shape: 同 Button 组件的shape Jan 13, 2025
@renaesop renaesop requested a review from Copilot January 22, 2025 06:36

Choose a reason for hiding this comment

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

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

Files not reviewed (2)
  • package.json: Language not supported
  • docs/api/shineout/dropdown.ts: Evaluated as low risk
Comments suppressed due to low confidence (1)

packages/shineout/src/dropdown/example/t-002-no-placeholder.tsx:45

  • The property name buttonShape should be consistent with the shape property used in the code.
<Dropdown data={data} placeholder={moreIcon} buttonShape="circle" hideArrow />
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