Merge pull request #17 from brianlovin/lighthouse

Fix accessibility issues
This commit is contained in:
Brian Lovin 2019-01-14 09:55:22 -08:00 committed by GitHub
commit 1472a8b9aa
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 9 additions and 2 deletions

View file

@ -72,7 +72,7 @@ export const AppRow = ({ app }: Props) => {
return (
<AppRowContainer href={app.url} target="_blank" rel="noopener noreferrer">
<AppMeta>
{app.image && <AppIcon src={app.image} />}
{app.image && <AppIcon alt={app.name} src={app.image} />}
<AppName>{app.name}</AppName>
</AppMeta>
{sourcesKeys && (

View file

@ -1,7 +1,7 @@
// @flow
import * as React from 'react';
import Link from 'next/link';
import { Container, ButtonRowContainer } from './style';
import { Container, ButtonRowContainer, Label } from './style';
import { PrimaryButton, GhostButton } from '../Button';
import Logo from './Logo';
@ -16,6 +16,7 @@ export default function Header(props: Props) {
<Container showHeaderShadow={showHeaderShadow} data-cy="header">
<Link href="/">
<a style={{ display: 'flex', alignItems: 'center' }}>
<Label>Security Checklist</Label>
<Logo />
</a>
</Link>

View file

@ -42,3 +42,9 @@ export const ButtonRowContainer = styled.div`
grid-area: actions;
align-items: center;
`;
export const Label = styled.h1`
position: absolute;
left: -9999px;
visibility: none;
`;