Skip to content

Conversation

@superRaytin
Copy link
Contributor

@superRaytin superRaytin commented Apr 25, 2016

MenuItem/SubMenu 增加 onMouseEnter/onMouseLeave 事件,只在 mode: inline 时生效。

ant-design/ant-design#1445

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 61ab5ab on superRaytin:feature/mouse-event-on-submenu-and-menuitem into * on react-component:master*.

src/MenuItem.jsx Outdated
}
}, 30);
if (props.mode === 'inline') {
if (props.onMouseLeave) {
Copy link
Member

Choose a reason for hiding this comment

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

已经有 defaultProps 了,不需要再做检测了吧。。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

嗯。。我修改一下

@afc163
Copy link
Member

afc163 commented Apr 25, 2016

  1. 所有模式都支持如何?
  2. 顺便补充上其他事件。react-component/cascader@b75541c#diff-af6343eac544a21d4dce3ee9c3ce19c5L105

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 2b173f8 on superRaytin:feature/mouse-event-on-submenu-and-menuitem into * on react-component:master*.

@superRaytin
Copy link
Contributor Author

@afc163 vertical 和 horizonal 的子菜单是 popup,支持的话会不会有点混乱。。

其他事件是指所有的 Mouse Events 吗?
https://facebook.github.io/react/docs/events.html#mouse-events

@afc163
Copy link
Member

afc163 commented Apr 25, 2016

也不算支持,算是修复所有原生事件传递。

@superRaytin
Copy link
Contributor Author

修复所有原生事件传递

那所有 rc 组件都需要吧,需要通盘来考虑 @yiminghe

@yiminghe yiminghe merged commit 27daa63 into react-component:master Apr 27, 2016
@yiminghe
Copy link
Member

只加必要的就行了

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.

5 participants