Fix: swap anchor with button to login at 'VehicleDetail.jsx' and upda…

…te styles
ITurres · Feb 2, 2024 · 7730bf1 · 7730bf1
1 parent 4a19f9c
commit 7730bf1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/features/vehicles/VehicleDetail.jsx b/src/features/vehicles/VehicleDetail.jsx
@@ -105,9 +105,9 @@ const VehicleDetail = () => {
                   Reserve
                 </Link>
               ) : (
-                <a href="#login" onClick={() => handleLoginPanel()}>
+                <button type="button" onClick={() => handleLoginPanel()}>
                   Login
-                </a>
+                </button>
               )}
             </div>
           </div>

diff --git a/src/styles/features/vehicles/VehicleDetail.scss b/src/styles/features/vehicles/VehicleDetail.scss
@@ -157,13 +157,15 @@ $screen-min-width: 992px;
     font: bold 2rem $heading-font;
   }
 
-  a {
+  button {
     padding: 10px 20px;
     color: $main-color;
     text-decoration: none;
     font: bold 1.5rem $heading-font;
     background-color: $secondary-main-color;
+    border: none;
     border-radius: 10px;
+    cursor: pointer;
   }
 
   @media only screen and (min-width: $screen-min-width) {
-Original file line number
+Diff line change
@@ Expand Up / @@ -105,9 +105,9 @@ const VehicleDetail = () => { @@
                       Reserve
                     </Link>
                   ) : (
-                    <a href="#login" onClick={() => handleLoginPanel()}>
+                    <button type="button" onClick={() => handleLoginPanel()}>
                       Login
-                    </a>
+                    </button>
                   )}
                 </div>
               </div>
@@ Expand Down @@