From baa84af0c5fdddb5a0f201e67db9313f1cd3c301 Mon Sep 17 00:00:00 2001
From: Rohan Kumar <seirdy@seirdy.one>
Date: Sat, 23 Apr 2022 20:21:02 -0700
Subject: [PATCH] Remove skip link, typo.

The skip link is unnecessary on my site since landmarks and headings
both allow skipping repeated content.
---
 assets/css/dark.css          |  3 +--
 assets/css/main.css          | 33 ++-------------------------------
 layouts/partials/header.html |  1 -
 3 files changed, 3 insertions(+), 34 deletions(-)

diff --git a/assets/css/dark.css b/assets/css/dark.css
index 421db2d..95818cb 100644
--- a/assets/css/dark.css
+++ b/assets/css/dark.css
@@ -4,8 +4,7 @@
 		color: #eee;
 	}
 
-	html,
-	.skip {
+	html {
 		/* "background" is short for a bunch of unnecessary CSS rules
 		 * background-color is all I need */
 		background-color: #0f0f0b;
diff --git a/assets/css/main.css b/assets/css/main.css
index c37a2f8..f84b6ef 100644
--- a/assets/css/main.css
+++ b/assets/css/main.css
@@ -3,11 +3,7 @@
  * everything else addresses a specific a11y, compatibility, or critical
  * usability need.
  * I also don't define any non-semantic classes besides unstyled-list.
- * The the other classnames referenced here are from microformats2. */
-
-/* Start with browser defaults: default fonts, non-dark-mode colors, etc
- * Only change things to solve a specific a11y or significant usability
- * need. One exception: cosmetic improvement for unstyled-list */
+ * The the other classnames referenced here are from microformats2 */
 
 html {
 	/* Mobile screens benefit from greater line-spacing so links are
@@ -54,7 +50,7 @@ blockquote {
 }
 
 /* narrow screens: reduce list indentation */
-dd,
+dl,
 ol,
 ul {
 	margin: .5em 0 0 0;
@@ -96,7 +92,6 @@ kbd {
 }
 
 /* narrow screens: remove unused figure margins */
-
 figure {
 	margin: 1em 0;
 }
@@ -213,30 +208,6 @@ a:focus,
 	outline: 3px solid;
 }
 
-/* Skip link is hidden until focused.
- * Needs background color so it can appear over other elements on narrow
- * screens. If we set a background color, we must also set a foreground
- * color. System colors like Canvas and LinkText aren't supported by all
- * browsers, so we make up our own. */
-.skip {
-	/* skip-link could appear on top of existing content, so it needs a
-	 * solid background to be legible. If we set the background, we must
-	 * also set the foreground. Not all browsers support system colors
-	 * ("Canvas", "LinkText"), so we need to pick our own colors.
-	 * #06c is the default link color in FF, IE, and ux.redhat.com
-	 * #00e is the default in WebKit, Blink, and ancient FF. */
-	background-color: #fff;
-	color: #00e;
-
-	position: absolute;
-	left: .25em;
-	top: -2em;
-}
-
-.skip:focus {
-	top: .5em;
-}
-
 /* Todo:
  * - Some browsers don't scale SVGs properly; the img container
  *   dimensions crop the image rather than scale it. Investigate
diff --git a/layouts/partials/header.html b/layouts/partials/header.html
index 4638411..9afac1a 100644
--- a/layouts/partials/header.html
+++ b/layouts/partials/header.html
@@ -1,5 +1,4 @@
 <header id="banner" itemscope itemtype="https://schema.org/WPHeader">
-	<a href="#content" class="skip">Skip to content</a>
 	<nav aria-label="Global">
 		<ul class="unstyled-list">
 			{{- $currentPage := . -}}