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

IE11 & 低于版本14的Edge浏览器, Tab键焦点落在图标上 #12524

Merged
merged 1 commit into from Oct 10, 2018

Conversation

@siyu77
Copy link
Contributor

@siyu77 siyu77 commented Oct 6, 2018

浏览器: IE 11, Edge
重现步骤: FormItem设置hasFeedback属性, 校验错误时显示图标. 在表单区域内使用tab键切换焦点.
预期: 焦点从一个输入框切换到下一个输入框
实际: 焦点从输入框切换到图标, 再切换到下一个输入框

参考: https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/8090208/

@zombieJ zombieJ requested a review from HeskeyBaozi Oct 8, 2018
Copy link
Contributor

@HeskeyBaozi HeskeyBaozi left a comment

approved.

@zombieJ zombieJ merged commit 9e72ca1 into ant-design:master Oct 10, 2018
13 of 16 checks passed
13 of 16 checks passed
ci/circleci: test_dom Your tests failed on CircleCI
Details
ci/circleci: test_es Your tests failed on CircleCI
Details
ci/circleci: test_lib Your tests failed on CircleCI
Details
WIP ready for review
Details
ci/circleci: compile Your tests passed on CircleCI!
Details
ci/circleci: dist Your tests passed on CircleCI!
Details
ci/circleci: lint Your tests passed on CircleCI!
Details
ci/circleci: setup Your tests passed on CircleCI!
Details
ci/circleci: test_dist Your tests passed on CircleCI!
Details
ci/circleci: test_dist_15 Your tests passed on CircleCI!
Details
ci/circleci: test_dom_15 Your tests passed on CircleCI!
Details
ci/circleci: test_es_15 Your tests passed on CircleCI!
Details
ci/circleci: test_lib_15 Your tests passed on CircleCI!
Details
ci/circleci: test_node Your tests passed on CircleCI!
Details
ci/circleci: test_node_15 Your tests passed on CircleCI!
Details
security/snyk - package.json (paranoidjk) No manifest changes detected
@afc163 afc163 mentioned this pull request Oct 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants