Wrap in button to satisy linter

opentripplanner · Oct 16, 2024 · aa67130 · aa67130
1 parent 2e9a65d
commit aa67130
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/client/src/components/SearchBar/SwapLocationsButton.tsx b/client/src/components/SearchBar/SwapLocationsButton.tsx
@@ -10,13 +10,17 @@ export function SwapLocationsButton({
   tripQueryVariables: TripQueryVariables;
   setTripQueryVariables: (tripQueryVariables: TripQueryVariables) => void;
 }) {
-  const onChange = () => {
+  const swapFromTo = () => {
     setTripQueryVariables({
       ...tripQueryVariables,
       from: tripQueryVariables.to,
       to: tripQueryVariables.from,
     });
   };
 
-  return <img className="swap-from-to" alt={HINT} title={HINT} src={swapImg} onClick={onChange} />;
+  return (
+    <button className="swap-from-to" onClick={swapFromTo} title={HINT}>
+      <img alt={HINT} src={swapImg} />
+    </button>
+  );
 }
diff --git a/client/src/style.css b/client/src/style.css
@@ -68,8 +68,13 @@
 }
 
 .search-bar .swap-from-to {
+  border: none;
+  background: none;
+  margin: 30px 0 auto 0;
+}
+
+.search-bar .swap-from-to img {
   width: 15px;
-  margin: 37px 0 auto 0;
 }
 
 .itinerary-list-container {