Skip to content

Commit

Permalink
fix: Assume NavBar is present when calculating height of explorer ini…
Browse files Browse the repository at this point in the history
…tially. (#502)

* added analytics forunified nav bar

* branch init

* assume navbar present

* update with comment

Co-authored-by: atarashansky <[email protected]>
  • Loading branch information
atarashansky and atarashansky authored Oct 18, 2022
1 parent af587a4 commit 92cbcf0
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion client/src/components/framework/layout.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,9 @@ const Layout: React.FC<Props> = (props) => {
<div
style={{
display: "grid",
paddingTop: seamlessEnabled ? globals.HEADER_HEIGHT_PX : 0,
// `??` is used so when `seamlessEnabled` is `undefined` or `true`, so we can add padding to prevent
// unneeded scrollbar
paddingTop: seamlessEnabled ?? true ? globals.HEADER_HEIGHT_PX : 0,
gridTemplateColumns: `
[left-sidebar-start] ${globals.leftSidebarWidth + 1}px
[left-sidebar-end graph-start] auto
Expand Down

0 comments on commit 92cbcf0

Please sign in to comment.