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

cascader tigger 为 hover 且开启 nzChangeOnSelect 时,hover 父级选项时会触发选中事件 #1966

Closed
shuizhongxiong opened this issue Aug 10, 2018 · 6 comments
Assignees
Labels
💪 Enhancement Good First Issue Issues recommended for first-time contributor

Comments

@shuizhongxiong
Copy link
Contributor

shuizhongxiong commented Aug 10, 2018

Version

1.3.0

Environment

chrome

Reproduction link

https://stackblitz.com/edit/ng-zorro-antd-cascader?embed=1&file=src/app/app.component.ts

Steps to reproduce

What is expected?

当同时使用 [nzExpandTrigger]="'hover'" 及 [nzChangeOnSelect]="true" 配置时,鼠标滑过父级选项时展开次级菜单,鼠标点击父级选项时才会触发 ngModelChange 事件

What is actually happening?

当同时使用 [nzExpandTrigger]="'hover'" 及 [nzChangeOnSelect]="true" 配置时,鼠标滑过父级选项时会触发 ngModelChange 事件。

Other?

实际业务场景是有一个行业列表,一级行业下面又有二级行业,希望鼠标滑过左侧一级行业时展开右侧二级行业,点击可以选中一级行业或是二级行业,同时隐藏列表。

@wzhudev
Copy link
Member

wzhudev commented Aug 13, 2018

@shuizhongxiong Thanks for your feedback. Is this what you expect? Then, we will fix that soon!

@wzhudev wzhudev added the Good First Issue Issues recommended for first-time contributor label Aug 13, 2018
@wzhudev
Copy link
Member

wzhudev commented Aug 13, 2018

Hint for fixing this issue:

We already have code base for this. Just need to find the correct method and change some boolean values...

@shuizhongxiong
Copy link
Contributor Author

@wendzhue Yes, this is what I expected.Thank you very much for fixing it!

@wzhudev
Copy link
Member

wzhudev commented Aug 13, 2018

@shuizhongxiong Are you interested in fixing this? Any pr is welcome!

@shuizhongxiong
Copy link
Contributor Author

@wendzhue Ok, let me try it.

@lock
Copy link

lock bot commented Aug 19, 2019

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 Aug 19, 2019
hsuanxyz pushed a commit to hsuanxyz/ng-zorro-antd that referenced this issue Aug 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
💪 Enhancement Good First Issue Issues recommended for first-time contributor
Projects
None yet
Development

No branches or pull requests

2 participants