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

[Image] 为什么加载失败状态下不能触发onClick #2190

Open
xmsz-stu opened this issue Apr 22, 2024 · 4 comments
Open

[Image] 为什么加载失败状态下不能触发onClick #2190

xmsz-stu opened this issue Apr 22, 2024 · 4 comments
Labels
feature request Request new feature

Comments

@xmsz-stu
Copy link

Which Component Need Enhancement 期望新增功能的组件

  • Image

What does the proposed API look like 期望支持的API

失败状态下也能触发onClick

Feature Description 功能描述

现在失败状态下,无法触发onClick
但我的场景下,是希望可以触发onClick

Additional information 补充说明

我现在只能通过callback去实现,但是特别麻烦

@xmsz-stu xmsz-stu added the feature request Request new feature label Apr 22, 2024
@YyumeiZhang
Copy link
Collaborator

加载失败的时候触发 onClick 是处于什么业务需求呢?想看你那边是否会通过 event.target 去拿什么参数,比如 src 啥的

@xmsz-stu
Copy link
Author

加载失败的时候触发 onClick 是处于什么业务需求呢?想看你那边是否会通过 event.target 去拿什么参数,比如 src 啥的

比如重新尝试下载图片或者没有图片的可以引导用户点击下载

@YyumeiZhang
Copy link
Collaborator

加载失败的时候触发 onClick 是处于什么业务需求呢?想看你那边是否会通过 event.target 去拿什么参数,比如 src 啥的

比如重新尝试下载图片或者没有图片的可以引导用户点击下载

原来的 onClick 是直接通过 restProps 透传给 Img 节点,因此 loading/加载错误状态下会有一层 overlay 在 img 节点上方。因此loading / 加载错误状态下点击无法生效。

https://github.com/DouyinFE/semi-design/blob/v2.56.3/packages/semi-ui/image/image.tsx#L196

目前支持是在 overlay 层也处理点击事件,此时无法通过 onClick 的 event.target 拿到 src,这种支持方式符合预期吗?

@xmsz-stu
Copy link
Author

加载失败的时候触发 onClick 是处于什么业务需求呢?想看你那边是否会通过 event.target 去拿什么参数,比如 src 啥的

比如重新尝试下载图片或者没有图片的可以引导用户点击下载

原来的 onClick 是直接通过 restProps 透传给 Img 节点,因此 loading/加载错误状态下会有一层 overlay 在 img 节点上方。因此loading / 加载错误状态下点击无法生效。

https://github.com/DouyinFE/semi-design/blob/v2.56.3/packages/semi-ui/image/image.tsx#L196

目前支持是在 overlay 层也处理点击事件,此时无法通过 onClick 的 event.target 拿到 src,这种支持方式符合预期吗?

我的场景符合

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

No branches or pull requests

2 participants