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.

Sign up for GitHub

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

Jump to bottom

Menu组件mode为vertical的时候，子菜单没有弹出 #3783

Closed
104gogo opened this issue Nov 10, 2016 · 8 comments · Fixed by #4609
Closed

Menu组件mode为vertical的时候，子菜单没有弹出 #3783

104gogo opened this issue Nov 10, 2016 · 8 comments · Fixed by #4609
Labels
help wanted The suggestion or request has been accepted, we need you to help us by sending a pull request.

Comments

@104gogo
Copy link

104gogo commented Nov 10, 2016

发生问题的环境是：

  • antd 版本: 1.5.1
  • 操作系统及其版本: win7
  • 浏览器及其版本: chrome 54.0.2840.87 m (64-bit)

您做了什么？请提供尽可能详细的重现步骤。

我想改变openKeys和selectedKeys来控制Menu的展示
当mode是inline的时候是正常的
但mode为vertical的时候，子菜单就不弹出了

可重现的在线演示

http://codepen.io/skyriver9/pen/MbaRWE?editors=0010
@yesmeck yesmeck added the 🐛 Bug Ant Design Team had proved that this is a bug. label Nov 10, 2016
@yesmeck yesmeck self-assigned this Nov 10, 2016
@yesmeck
Copy link
Member

yesmeck commented Nov 10, 2016

@afc163 https://github.com/ant-design/ant-design/blob/master/components/menu/index.tsx#L134-L136 这个好像是故意的？

@yesmeck
Copy link
Member

yesmeck commented Nov 10, 2016

@104gogo 不知道你的使用场景是？

@yesmeck yesmeck removed the 🐛 Bug Ant Design Team had proved that this is a bug. label Nov 10, 2016
@104gogo
Copy link
Author

104gogo commented Nov 10, 2016

@yesmeck router变了显示对应的菜单

@benjycui
Copy link
Contributor

#3419

@benjycui benjycui added the help wanted The suggestion or request has been accepted, we need you to help us by sending a pull request. label Nov 10, 2016
@yesmeck yesmeck removed their assignment Nov 10, 2016
@yesmeck yesmeck added 🐛 Bug Ant Design Team had proved that this is a bug. Enhancement and removed 🐛 Bug Ant Design Team had proved that this is a bug. labels Dec 9, 2016
afc163 added a commit that referenced this issue Jan 15, 2017
@afc163
Fix openKeys and defaultOpenKeys of Menu, close #3783
c00f582
@afc163 afc163 mentioned this issue Jan 15, 2017
Merged
@afc163
Copy link
Member

afc163 commented Jan 15, 2017

一开始确实没有需求，就偷懒了，在上面 PR 里一并修复了。

@afc163
Copy link
Member

afc163 commented Jan 15, 2017

陈年老 issue ... 响应得慢了请多谅解。

@104gogo
Copy link
Author

104gogo commented Jan 16, 2017

@afc163 good job!

yesmeck pushed a commit that referenced this issue Jan 16, 2017
@afc163 @yesmeck
Fix openKeys and defaultOpenKeys of Menu, close #3783 (#4609)
0a68959
@benjycui benjycui mentioned this issue Feb 15, 2017
Closed
@afc163 afc163 mentioned this issue Feb 15, 2017
Closed
@lock
Copy link

lock bot commented May 2, 2018

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked as resolved and limited conversation to collaborators May 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
help wanted The suggestion or request has been accepted, we need you to help us by sending a pull request.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix openKeys and defaultOpenKeys of Menu ant-design/ant-design
4 participants
@yesmeck @afc163 @benjycui @104gogo