From 710e1e937f88dfab5224904406417b69f2e9094a Mon Sep 17 00:00:00 2001
From: marc tobias <mtmail@gmx.net>
Date: Tue, 2 Feb 2021 02:36:02 +0100
Subject: [PATCH] fix remaining compile warnings

---
 TODO.md                           |  1 -
 src/components/MapPosition.svelte |  2 +-
 src/components/SearchBar.svelte   | 14 ++++++++------
 3 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/TODO.md b/TODO.md
index ecc7c4a..02c87a8 100644
--- a/TODO.md
+++ b/TODO.md
@@ -16,7 +16,6 @@
 
 ## Code cleanup
 
-* fix all a11y warnings during `yarn build`
 * move all bootstrap CSS overwrite rules into a bootstrap.theme.css file
 * move TODO list into github issues
 * aResults vs aPlace vs aFeature variable naming
diff --git a/src/components/MapPosition.svelte b/src/components/MapPosition.svelte
index 2d6376e..13e401a 100644
--- a/src/components/MapPosition.svelte
+++ b/src/components/MapPosition.svelte
@@ -94,7 +94,7 @@
     <br>
     mouse position: {mouse_position}
   </div>
-  <div id="map-position-close"><a href="#" on:click={handleHideClick}>hide</a></div>
+  <div id="map-position-close"><a href="#hide" on:click={handleHideClick}>hide</a></div>
 </div>
 
 
diff --git a/src/components/SearchBar.svelte b/src/components/SearchBar.svelte
index 507bffc..a2a8b72 100644
--- a/src/components/SearchBar.svelte
+++ b/src/components/SearchBar.svelte
@@ -92,8 +92,9 @@
     <form class="form-inline" role="search" accept-charset="UTF-8" action="">
       <div class="form-group">
         <input name="format" type="hidden" value="html">
-        <label>lat</label>
-        <input name="lat"
+        <label for="reverse-lat">lat</label>
+        <input id="reverse-lat"
+               name="lat"
                type="text"
                class="form-control form-control-sm"
                placeholder="latitude"
@@ -102,14 +103,15 @@
            on:click|preventDefault|stopPropagation={handleSwitchCoords}
            class="btn btn-outline-secondary btn-sm"
            title="switch lat and lon">&lt;&gt;</a>
-        <label>lon</label>
-        <input name="lon"
+        <label for="reverse-lon">lon</label>
+        <input id="reverse-lon"
+               name="lon"
                type="text"
                class="form-control form-control-sm"
                placeholder="longitude"
                value="{api_request_params.lon || ''}" />
-        <label>max zoom</label>
-        <select name="zoom" class="form-control form-control-sm" value="{api_request_params.zoom}">
+        <label for="reverse-zoom">max zoom</label>
+        <select id="reverse-zoom" name="zoom" class="form-control form-control-sm" value="{api_request_params.zoom}">
           <option value="" selected={!api_request_params.zoom}>---</option>
           {#each zoomLevels() as zoomTitle, i}
             <option value="{i}" selected={i === api_request_params.zoom}>{i} - {zoomTitle}</option>
-- 
2.39.5