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

style: Fix the problem of wrong style of ButtonGroup with theme as ou… #2328

Merged
merged 2 commits into from
Jul 5, 2024

Conversation

YyumeiZhang
Copy link
Collaborator

…tline

中文模板 / Chinese Template

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Test Case
  • TypeScript definition update
  • Document improve
  • CI/CD improve
  • Branch sync
  • Other, please describe:

PR description

Fixes #
修复前
image
修复后
image
修复后鼠标hover 高亮
image

Changelog

🇨🇳 Chinese

  • Style: 修复 theme 为 outline 的 ButtonGroup 样式错误问题

🇺🇸 English

  • Style: Fix the problem of wrong style of ButtonGroup with theme as outline

Checklist

  • Test or no need
  • Document or no need
  • Changelog or no need

Other

  • Skip Changelog

Additional information

Copy link

codesandbox-ci bot commented Jul 3, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 859cb1b:

Sandbox Source
pr-story Configuration

Copy link

cypress bot commented Jul 3, 2024

3 failed tests on run #2612 ↗︎

3 260 10 0 Flakiness 0
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.

Details:

Merge 064b76a into 649dc89...
Project: semi-design Commit: 485968ffad ℹ️
Status: Failed Duration: 10:45 💡
Started: Jul 3, 2024 6:42 AM Ended: Jul 3, 2024 6:53 AM

Review all test suite changes for PR #2328 ↗︎

@YyumeiZhang YyumeiZhang merged commit f29429f into main Jul 5, 2024
5 of 7 checks passed
@YyumeiZhang YyumeiZhang deleted the fix/button-group-outline branch July 5, 2024 06:08
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