Add "Skip to Main Content" link #1635
candywangAnnotations
6 errors and 1 warning
|
Test:
src/components/layout.tsx#L13
`./Layout.module.css` import should occur before import of `./grid-aware/Footer`
|
|
Test:
src/components/layout.tsx#L53
Replace `Skip·to·Main·Content` with `⏎··········Skip·to·Main·Content⏎········`
|
|
Test:
src/components/layout.tsx#L59
Anchors must have content and the content must be accessible by a screen reader
|
|
Test:
src/components/layout.tsx#L59
The href attribute is required for an anchor to be keyboard accessible. Provide a valid, navigable address as the href value. If you cannot provide an href, but still need the element to resemble a link, use a button and change it with appropriate styles. Learn more: https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/HEAD/docs/rules/anchor-is-valid.md
|
|
Test:
src/components/layout.tsx#L59
Empty components are self-closing
|
|
Test
Process completed with exit code 1.
|
|
Test
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|