ng-zorro-antd icon indicating copy to clipboard operation
ng-zorro-antd copied to clipboard

Setps组件,当前活跃Setp的点击事件不应当被过滤掉

Open xingkoo opened this issue 3 years ago • 6 comments

Reproduction link

https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/components/steps/step.component.ts#L186

Steps to reproduce

当前活跃的步骤,点击事件无效,被过滤掉了。重现链接指向的代码可见

What is expected?

至少在使用导航模式时,应当响应点击事件,以便做一些响应处理。

What is actually happening?

现在活跃的步骤被拦截掉了。

Environment Info
ng-zorro-antd 13.2.2
Browser chrome

xingkoo avatar Jun 14 '22 16:06 xingkoo

Translation of this issue:

Setps component, the current active setp clicks should not be filtered out

REPRODUCTION LINK

[https://github.com/ng-zorro/ng-norro-Ntd/blob/master/components/step.component.ts#l186] (https://github.com/ng-zorro/ng- Zorro-ATD/BLOB/Master/Components/Steps/Step.comPonent.ts#L186)

STEPS To Reproduce

The current active steps, clicking the event is invalid and was filtered out. It can be visible to the code pointed to the link

What is exfected?

At least when using the navigation mode, the click event should be responded to do some response.

What is actually happy?

The active steps are now intercepted.

| ENVIRONMENT | Info | | --- | --- | | | NG-Zorro-ATD | 13.2.2 | | Browser | Chrome |

zorro-bot[bot] avatar Jun 14 '22 16:06 zorro-bot[bot]

目前 ng-zorro-antd 的行为和 ant-design 会基本保持一致,当前 step 点击认为是无变更,所以不会重复响应事件。可以描述下你需要响应的使用场景方便我们评估是否应该增加 :)

simplejason avatar Jun 16 '22 02:06 simplejason

一、 在一些每个进程需要展示过多信息的情况下,需要以类似tabs的方式存在, 以分开显示每个环节的内容,此时,我不能回显当前进度的信息。

二、 使用竖排进程,每个进程的内容很多,页面很长。 因为我固定了一个横排的导航模式的进程条,和下面的竖排实际业务进程度进程一致,点击每个进程,跳到对应的进程的锚点。 点击其他进程时,正常跳转过去,但是当前进程,无法跳到。

xingkoo avatar Jun 16 '22 10:06 xingkoo

我认为可以增加一个当前进程是否生效的开关来决定是否过滤该事件,会是一个更好的解决方案。

xingkoo avatar Jun 16 '22 10:06 xingkoo

这个问题可以解决吗?因为目前NG-ZORRO是写死的,没有留接口,导致自己想单独调整这个位置也不方便调整。

xingkoo avatar Jul 27 '22 13:07 xingkoo

@simplejason 简单来说,就是内容过多,将setps以导航方式使用,或者单个步骤内容以选项卡表示时,需要通过点击选项来调整滚动条位置或者切换当前步骤对应的选择卡。而此时『当前步骤』的点击事件被过滤掉了。 实际上,最合适的是给予一个开关,可以默认不响应,但需要时,可以开启。以适应不同的场景。

xingkoo avatar Jul 27 '22 13:07 xingkoo