Skip to content

Commit

Permalink
Fix accessibility issues
Browse files Browse the repository at this point in the history
  • Loading branch information
evadecker committed May 27, 2024
1 parent e29f804 commit f7ea8ae
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 52 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
"build": "astro check && astro build",
"preview": "astro preview",
"astro": "astro",
"test": "playwright test --ui"
"test": "playwright test --reporter=html"
},
"dependencies": {
"@astrojs/check": "^0.7.0",
Expand Down
50 changes: 0 additions & 50 deletions playwright.config.ts
Original file line number Diff line number Diff line change
@@ -1,74 +1,24 @@
import { defineConfig, devices } from '@playwright/test';

/**
* Read environment variables from file.
* https://github.com/motdotla/dotenv
*/
// require('dotenv').config();

/**
* See https://playwright.dev/docs/test-configuration.
*/
export default defineConfig({
testDir: './tests',
/* Run tests in files in parallel */
fullyParallel: true,
/* Fail the build on CI if you accidentally left test.only in the source code. */
forbidOnly: !!process.env.CI,
/* Retry on CI only */
retries: process.env.CI ? 2 : 0,
/* Opt out of parallel tests on CI. */
workers: process.env.CI ? 1 : undefined,
/* Reporter to use. See https://playwright.dev/docs/test-reporters */
reporter: 'html',
/* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
use: {
/* Base URL to use in actions like `await page.goto('/')`. */
baseURL: 'http://localhost:4321',

/* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */
trace: 'on-first-retry',
},

/* Configure projects for major browsers */
projects: [
{
name: 'chromium',
use: { ...devices['Desktop Chrome'] },
},

{
name: 'firefox',
use: { ...devices['Desktop Firefox'] },
},

{
name: 'webkit',
use: { ...devices['Desktop Safari'] },
},

/* Test against mobile viewports. */
// {
// name: 'Mobile Chrome',
// use: { ...devices['Pixel 5'] },
// },
// {
// name: 'Mobile Safari',
// use: { ...devices['iPhone 12'] },
// },

/* Test against branded browsers. */
// {
// name: 'Microsoft Edge',
// use: { ...devices['Desktop Edge'], channel: 'msedge' },
// },
// {
// name: 'Google Chrome',
// use: { ...devices['Desktop Chrome'], channel: 'chrome' },
// },
],

/* Run your local dev server before starting the tests */
webServer: {
command: 'pnpm dev',
url: 'http://localhost:4321',
Expand Down
4 changes: 3 additions & 1 deletion src/pages/index.astro
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ const year = new Date().getFullYear();
description="Namesake streamlines the legal name and gender marker change process for trans, nonbinary, and gender expansive folks."
>
<main>
<a href="/"><Logo /></a>
<a href="/" aria-label="Namesake homepage"><Logo /></a>
<div class="hero">
<div class="contents">
<h1>Be seen for who you are.</h1>
Expand Down Expand Up @@ -47,10 +47,12 @@ const year = new Date().getFullYear();
<div class="social">
<a
href="https://www.instagram.com/joinnamesake/"
aria-label="View Namesake on Instagram"
title="View Namesake on Instagram"><RiInstagramLine size="24px" /></a
>
<a
href="https://www.linkedin.com/company/namesake-collaborative/"
aria-label="View Namesake on LinkedIn"
title="View Namesake on LinkedIn"><RiLinkedinLine size="24px" /></a
>
</div>
Expand Down

0 comments on commit f7ea8ae

Please sign in to comment.