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

fix: remove aria-hidden #6688

Merged

Conversation

Layouwen
Copy link
Member

修复 ImageUploader 点击的 aria-hidden 报错 #6684

Copy link
Contributor

PR preview has been successfully built and deployed to https://antd-mobile-preview-pr-6688.surge.sh

@@ -445,7 +445,7 @@ exports[`ImageUploader upload status 1`] = `
</span>
<input
accept="image/*"
aria-hidden="true"
aria-label="adm-image-uploader-input"
Copy link
Member

Choose a reason for hiding this comment

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

这个 label 不要默认填,可以接受 aria-label 属性来让开发者填。prefix 不是给拼label 用的。而且这也会对多语言用户产生困扰~

@Layouwen Layouwen force-pushed the fix/image-uploader-aria-hidden-error branch from 1c43648 to b0c9a99 Compare July 31, 2024 10:09
@zombieJ zombieJ merged commit 2ec5d58 into ant-design:master Jul 31, 2024
5 checks passed
@Layouwen Layouwen deleted the fix/image-uploader-aria-hidden-error branch August 1, 2024 00:54
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.

2 participants