-
-
Notifications
You must be signed in to change notification settings - Fork 46.9k
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: #24519 make upload action buttons focusable #24604
fix: #24519 make upload action buttons focusable #24604
Conversation
CI failed with message:
|
title={locale.removeFile} | ||
onClick={() => this.handleClose(file)} | ||
tabIndex={0} | ||
/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
包一个 a 或者 button 标签应该就行了,加 tabIndex 是下策。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
我一开始也是这么想的,但是怕改结构你们会有什么 concern ,所以就选 tabindex 这种改动比较小的方案
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems you are right, 我选上以后没法删除...
changelog 要从用户角度写,对于这个问题应该是:
我手动先修改了,下次的 PR 可以注意一下。 |
[中文版模板 / Chinese template]
close #24519
🤔 This is a ...
🔗 Related issue link
#24519
💡 Background and solution
tabindex
on icon to0
opacity
control to action-icon📝 Changelog
☑️ Self Check before Merge