fix(frontend): remove duplicate id for searchbars (#3000)

Only one element should have the same id. The id is used to link with the label's `for` attribute. There may be similar issues other places, but this is the one that was brought to my attention. Resolves #2999
modrinth · Dec 10, 2024 · 7d480ff · 7d480ff
1 parent d851f32
commit 7d480ff
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/apps/frontend/src/pages/search/[searchProjectType].vue b/apps/frontend/src/pages/search/[searchProjectType].vue
@@ -48,12 +48,12 @@
       <section class="card gap-1" :class="{ 'max-lg:!hidden': !sidebarMenuOpen }">
         <div class="flex items-center gap-2">
           <div class="iconified-input w-full">
-            <label class="hidden" for="search">Search</label>
+            <label class="hidden" for="filter-search">Search</label>
             <SearchIcon aria-hidden="true" />
             <input
-              id="search"
+              id="filter-search"
               v-model="queryFilter"
-              name="search"
+              name="filter-search"
               type="search"
               placeholder="Search filters..."
               autocomplete="off"
@@ -174,13 +174,13 @@
             Filters...
           </button>
           <div class="iconified-input">
-            <label class="hidden" for="search">Search</label>
+            <label class="hidden" for="project-search">Search</label>
             <SearchIcon aria-hidden="true" />
             <input
-              id="search"
+              id="project-search"
               v-model="query"
               type="search"
-              name="search"
+              name="project-search"
               :placeholder="`Search ${projectType.display}s...`"
               autocomplete="off"
               @input="onSearchChange(1)"