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

Radio.Group中的Radio.Button覆盖了Badge #13132

Closed
ybning opened this Issue Nov 14, 2018 · 3 comments

Comments

Projects
None yet
4 participants
@ybning

ybning commented Nov 14, 2018

  • I have searched the issues of this repository and believe that this is not a duplicate.

Version

3.8.0

Environment

Win7 64位,Chrome 66.0.3359.117

Reproduction link

Edit on CodeSandbox

Steps to reproduce

11111111111
打开重现链接 https://codesandbox.io/s/ll4o286l2l 即可看到效果

What is expected?

Radio.Group中的Radio.Button不覆盖Badge

What is actually happening?

Radio.Group中的Radio.Button覆盖了Badge

@zy410419243

This comment has been minimized.

Collaborator

zy410419243 commented Nov 14, 2018

可以加个style先 这玩意涉及好广,PR 丢人了

<Badge count="专属" offset={[10, -10]} style={{ zIndex: 1 }}>
    aaaaa
</Badge>

ztplz added a commit to ztplz/ant-design that referenced this issue Nov 14, 2018

@ztplz ztplz referenced this issue Nov 14, 2018

Merged

Fix #13132 #13143

zombieJ added a commit that referenced this issue Nov 14, 2018

@afc163 afc163 reopened this Nov 15, 2018

@afc163

This comment has been minimized.

Member

afc163 commented Nov 15, 2018

afc163 added a commit that referenced this issue Nov 15, 2018

Merge branch 'master' into feature
* master:
  Add missing types for tree
  fix: notification's description should be optional
  Fix #13132 (#13143)
@zombieJ

This comment has been minimized.

Member

zombieJ commented Nov 26, 2018

Seems works well. Issue close:
kapture 2018-11-26 at 14 32 07

@zombieJ zombieJ closed this Nov 26, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment