Skip to content

Commit

Permalink
Merge pull request #28 from MFB-Technologies-Inc/feature/24-add-a-tit…
Browse files Browse the repository at this point in the history
…le-to-the-spinner-image

Feature/24 add a title to the spinner image
  • Loading branch information
lemke-ethan authored Nov 3, 2023
2 parents 9836abd + 4737806 commit 9195f80
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
4 changes: 0 additions & 4 deletions src/components/LoadingSpinner.scss
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,4 @@ $half-size: math.div($size, 2);
transform: rotate(350deg);
}
}

.alt-text {
display: none;
}
}
4 changes: 3 additions & 1 deletion src/components/LoadingSpinner.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,10 @@ export const LoadingSpinner: React.FC = () => (
y="0px"
viewBox="0 0 144 144"
xmlSpace="preserve"
aria-labelledby="loadingSpinnerTitle"
role="img"
>
<title id="loadingSpinnerTitle">Loading spinner</title>
<g>
<path
opacity="0.9"
Expand Down Expand Up @@ -58,6 +61,5 @@ export const LoadingSpinner: React.FC = () => (
/>
</g>
</svg>
<span className="alt-text">Loading...</span>
</div>
)

0 comments on commit 9195f80

Please sign in to comment.