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

menu 组件的箭头样式 #20035

Closed
1 task done
ohhoney1 opened this issue Dec 2, 2019 · 6 comments · Fixed by #27565
Closed
1 task done

menu 组件的箭头样式 #20035

ohhoney1 opened this issue Dec 2, 2019 · 6 comments · Fixed by #27565
Assignees
Labels

Comments

@ohhoney1
Copy link
Contributor

ohhoney1 commented Dec 2, 2019

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

Reproduction link

https://ant.design/components/menu-cn/

Steps to reproduce

查看menu组件的箭头样式问题

What is expected?

正常显示

What is actually happening?

  1. chrome :箭头向上时,显示有个小圆点,并不是真正的箭头,请参考图片1。
  2. safari :箭头向下时,显示有问题,请参考图片2。

简单调试了下,因为不同浏览器对有小数点像素的渲染不同,components/menu/style/index.less#L233 这里的高度改为 2px ,在两个浏览器下显示都正常了

Environment Info
antd 3.26.0
React 16.9
System mac OS 10.14
Browser Chrome
@ohhoney1
Copy link
Contributor Author

ohhoney1 commented Dec 2, 2019

image

image

@afc163
Copy link
Member

afc163 commented Dec 3, 2019

@jljsj33 来看看?

afc163 added a commit that referenced this issue Nov 5, 2020
afc163 added a commit that referenced this issue Nov 6, 2020
* style: optimize menu expand icon color

close #20035

* feat: 🆕 support Menu expandIcon

* revert arrow color
@afc163
Copy link
Member

afc163 commented Nov 6, 2020

Fixed in #27565

@afc163 afc163 closed this as completed Nov 6, 2020
yoyo837 pushed a commit that referenced this issue Nov 19, 2020
* style: optimize menu expand icon color

close #20035

* feat: 🆕 support Menu expandIcon

* revert arrow color
@qinghua5453
Copy link

话说 有没有发现 这个属性overflowedIndicator 根本就不生效

@OWRIG
Copy link

OWRIG commented Feb 11, 2022

话说 有没有发现 这个属性overflowedIndicator 根本就不生效

+1

@Jasonlee6789
Copy link

请问下怎么把这个箭头从右边换到左边啊?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants