Skip to content

Commit

Permalink
Merge pull request #18 from nushydude/17-fix-missing-gap-between-head…
Browse files Browse the repository at this point in the history
…er-and-content-on-best-buy-and-best-dca-pages

Fix the gap between the header and the best buy pages
  • Loading branch information
nushydude authored Nov 4, 2023
2 parents 88130a3 + e8e2a1c commit 10b2071
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
2 changes: 1 addition & 1 deletion src/App.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ export const App = () => {

<Header />

<div className="w-100 max-w-7xl mx-auto px-2 flex flex-col flex-grow pt-18 sm:pt-12">
<div className="w-100 max-w-7xl mx-auto px-2 flex flex-col flex-grow pt-16 sm:pt-14">
<AppRoutes />
</div>

Expand Down
5 changes: 1 addition & 4 deletions src/components/Header.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,9 @@ const links = [
];

export const Header = () => {
const [ref, { x, y, width, height, top, right, bottom, left }] =
useMeasure<HTMLDivElement>();
const [ref, { height }] = useMeasure<HTMLDivElement>();
const { removeUser, isLoggedIn } = useContext(UserContext);

console.log('headerHeight', x, y, width, height, top, right, bottom, left);

return (
<div
data-testid="header"
Expand Down

0 comments on commit 10b2071

Please sign in to comment.