Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

[#2052] Fixed an accessibility issue with the recent bots list remove button #2091

Merged
tonyanziano merged 1 commit into from
Mar 5, 2020
Merged

[#2052] Fixed an accessibility issue with the recent bots list remove button #2091

tonyanziano merged 1 commit into from
Mar 5, 2020

Conversation

@tonyanziano
Copy link
Contributor

@tonyanziano tonyanziano commented Mar 5, 2020

#2052

===

Screen reader now reads the "X" button in the recent bots list as "Remove bot button"

@coveralls
Copy link

coveralls commented Mar 5, 2020

Coverage Status

Coverage remained the same at 68.154% when pulling c77db89 on toanzian/acc-#2052 into 5ca8253 on master.

@tonyanziano tonyanziano merged commit cb4d18f into master Mar 5, 2020
@tonyanziano tonyanziano deleted the toanzian/acc-#2052 branch March 5, 2020 20:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@a-b-r-o-w-n a-b-r-o-w-n Awaiting requested review from a-b-r-o-w-n

@compulim compulim Awaiting requested review from compulim

@cwhitten cwhitten Awaiting requested review from cwhitten

@tdurnford tdurnford Awaiting requested review from tdurnford

2 more reviewers

@srinaath srinaath srinaath approved these changes

@corinagum corinagum corinagum approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@tonyanziano @coveralls @srinaath @corinagum