Skip to content

Commit 65f7455

Browse files
authored
Merge pull request #407 from CodeForPhilly/web-app-narrow-header
decrease the height of the header bar so the user has more vertical space to work. use small logo instead of large logo in header.
2 parents a5f3957 + aff17b3 commit 65f7455

3 files changed

Lines changed: 30 additions & 5 deletions

File tree

Lines changed: 25 additions & 0 deletions
Loading

builder-frontend/src/components/Header/Header.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import { useAuth } from "../../context/AuthContext";
22
import { useLocation, useNavigate } from "@solidjs/router";
33
import { Component, For, Show } from "solid-js";
44

5-
import bdtLogo from "@/assets/logos/bdt-logo-large-mono-light.svg";
5+
import bdtLogo from "@/assets/logos/bdt-logo-small-mono-light.svg";
66
import { HamburgerMenu } from "@/components/shared/HamburgerMenu";
77

88
import "./Header.css";
@@ -19,7 +19,7 @@ const HeaderButton = ({
1919
<div
2020
onClick={onClick}
2121
class="
22-
px-4 py-3 text-md font-bold text-gray-700 rounded-md
22+
px-4 py-2 text-md font-bold text-gray-700 rounded-md
2323
flex items-center
2424
hover:bg-gray-300 cursor-pointer select-none"
2525
>
@@ -79,12 +79,12 @@ export default function Header() {
7979
};
8080

8181
return (
82-
<header class="bg-gray-200 min-h-24 h-24 px-4 flex items-center justify-between border-b-2 border-gray-300">
82+
<header class="bg-gray-200 min-h-12 h-12 px-4 flex items-center justify-between border-b-2 border-gray-300">
8383
<div class="flex items-center space-x-6">
8484
<img
8585
src={bdtLogo}
8686
alt="BDT logo"
87-
class="w-36 cursor-pointer"
87+
class="w-18 cursor-pointer"
8888
onClick={() => navigate("/")}
8989
/>
9090
</div>

builder-frontend/src/components/shared/HamburgerMenu/HamburgerMenu.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
}
77

88
.menu-toggle {
9-
@apply px-4 py-3 font-bold text-gray-700 rounded-sm;
9+
@apply px-4 py-2 font-bold text-gray-700 rounded-sm;
1010
}
1111
.menu-toggle:hover {
1212
@apply bg-gray-300 cursor-pointer;

0 commit comments

Comments
 (0)