Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

added aria-hidden to the action button icon #1778

Merged
merged 4 commits into from
Jul 2, 2018

Conversation

blackbaud-conorwright
Copy link
Contributor

added aria-hidden to the action button icon so that screen reader users do not receive unintelligible '?' boxes in addition to the button text

Resolves: #1466

@codecov-io
Copy link

codecov-io commented Jun 29, 2018

Codecov Report

Merging #1778 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1778   +/-   ##
=======================================
  Coverage   99.98%   99.98%           
=======================================
  Files         410      410           
  Lines        8542     8542           
  Branches     1253     1253           
=======================================
  Hits         8541     8541           
  Misses          1        1

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7d243e6...e0b808c. Read the comment docs.

@Blackbaud-SteveBrush Blackbaud-SteveBrush self-assigned this Jul 2, 2018
@Blackbaud-SteveBrush Blackbaud-SteveBrush merged commit c16ee63 into master Jul 2, 2018
@Blackbaud-SteveBrush Blackbaud-SteveBrush deleted the fix-action-button-icon-aria-hidden branch July 2, 2018 20:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants