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 missing alt-text #328
Fix missing alt-text #328
Conversation
Maybe a suggestion: adding react warnings to the travis build. |
Codecov Report
@@ Coverage Diff @@
## master #328 +/- ##
=======================================
Coverage 56.38% 56.38%
=======================================
Files 113 113
Lines 5627 5627
=======================================
Hits 3173 3173
Misses 2454 2454
Continue to review full report at Codecov.
|
I think this specific warning from |
@@ -288,7 +288,7 @@ decorators.Header = ({style, node}) => { | |||
decorators.Loading = ({style}) => { | |||
return ( | |||
<div style={style}> | |||
<img className="loading-spinner" src={spinner}/> | |||
<img alt="" className="loading-spinner" src={spinner}/> |
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.
Does it make sense to actually have some text here?
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.
Not really, it's a decorative spinner. The most useful alt-text would be 'loading' but that's unnecessary because the next word already is 'loading'.
Thanks for the contribution! |
No description provided.