From e7c70945e6f8d84a42cafd0b33ca091392065ea8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Florentin=20Schleu=C3=9F?= <git@macbrayne.de>
Date: Wed, 18 May 2022 03:31:58 +0200
Subject: [PATCH] Add skip links for "skip to main content" and "skip to search
 results" (#445)

---
 layouts/default.vue | 38 +++++++++++++++++++++++++++++++++++++-
 pages/search.vue    | 15 ++++++++++++++-
 2 files changed, 51 insertions(+), 2 deletions(-)

diff --git a/layouts/default.vue b/layouts/default.vue
index 1da77334b..899743502 100644
--- a/layouts/default.vue
+++ b/layouts/default.vue
@@ -2,6 +2,12 @@
   <div class="layout">
     <header class="site-header" role="presentation">
       <section class="navbar columns" role="navigation">
+        <section class="skip column" role="presentation">
+          <a href="#main">Skip to Main Content</a>
+          <a v-if="registeredSkipLink" :href="registeredSkipLink.id">{{
+            registeredSkipLink.text
+          }}</a>
+        </section>
         <section class="logo column" role="presentation">
           <NuxtLink to="/" aria-label="Modrinth home page">
             <ModrinthLogo aria-hidden="true" class="text-logo" />
@@ -225,7 +231,7 @@
         :max="5"
         :ignore-duplicates="true"
       />
-      <Nuxt />
+      <Nuxt id="main" />
     </main>
     <footer>
       <div class="logo-info" role="region" aria-label="Modrinth information">
@@ -333,6 +339,7 @@ export default {
       isDropdownOpen: false,
       version: process.env.version || 'unknown',
       isMobileMenuOpen: false,
+      registeredSkipLink: null,
     }
   },
   async fetch() {
@@ -363,6 +370,11 @@ export default {
       this.$router.push(this.$route.path)
     }
   },
+  created() {
+    this.$nuxt.$on('registerSkipLink', (data) => {
+      this.registeredSkipLink = data
+    })
+  },
   methods: {
     toggleMobileMenu() {
       window.scrollTo(0, 0)
@@ -406,6 +418,30 @@ export default {
 </script>
 
 <style lang="scss">
+.skip a {
+  clip: rect(1px, 1px, 1px, 1px);
+  height: 1px;
+  overflow: hidden;
+  position: absolute;
+  white-space: nowrap;
+  width: 1px;
+}
+
+.skip a:focus {
+  clip: auto;
+  height: auto;
+  overflow: auto;
+  position: absolute;
+  width: auto;
+  padding: 0.5rem 0.75rem;
+  background-color: var(--color-brand);
+  color: var(--color-brand-inverted);
+  border-radius: var(--size-rounded-max);
+  margin: 0 0.5rem 0 0;
+  box-shadow: inset 0px -1px 1px rgba(17, 24, 39, 0.1);
+  z-index: 1;
+}
+
 .layout {
   min-height: 100vh;
   background-color: var(--color-bg);
diff --git a/pages/search.vue b/pages/search.vue
index c0b3274a2..5145b33e4 100644
--- a/pages/search.vue
+++ b/pages/search.vue
@@ -228,7 +228,7 @@
           <LogoAnimated aria-hidden="true" />
           <p>Loading...</p>
         </div>
-        <div v-else role="list" aria-label="Search results">
+        <div v-else id="search-results" role="list" aria-label="Search results">
           <SearchResult
             v-for="result in results"
             :id="result.slug ? result.slug : result.project_id"
@@ -326,6 +326,8 @@ export default {
       maxResults: 20,
 
       sidebarMenuOpen: false,
+
+      skipLink: '#search-results',
     }
   },
   async fetch() {
@@ -395,6 +397,17 @@ export default {
       },
     },
   },
+  created() {
+    // This is currently using the global event bus as I couldn't figure out how to use the local one
+    this.$nuxt.$emit('registerSkipLink', {
+      id: '#search-results',
+      text: 'Skip to Search Results',
+    })
+  },
+  destroyed() {
+    // Not sure about this
+    this.$nuxt.$emit('registerSkipLink')
+  },
   methods: {
     async clearFilters() {
       for (const facet of [...this.facets]) await this.toggleFacet(facet, true)