Skip to content

Commit 72caf83

Browse files
committed
fix(header): fix desktop rendering after resize
1 parent dd7c6a8 commit 72caf83

File tree

2 files changed

+15
-4
lines changed

2 files changed

+15
-4
lines changed

components/ui/header/header.templ

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,23 @@ templ desktopHeader() {
5252

5353
templ Header(isMobile bool) {
5454
<header class={ "bg-accent", "p-[20px]", "md:p-[10px]" }>
55+
// mobiles are usually shitty at rendering, so this prevents mobiles from rendering two blocks and choosing one using CSS.
5556
if isMobile {
5657
@mobileHeader()
5758
} else {
58-
@desktopHeader()
59+
<div style="display: contents" class={ "hidden", "md:block" }>
60+
@desktopHeader()
61+
</div>
62+
<!--
63+
for when opened on a desktop and the client's width is really small,
64+
and a similar thing isn't being done for mobile, since mobile clients can't change width w/o reloading
65+
-->
66+
<div style="display: contents" class={ "md:hidden" }>
67+
@mobileHeader()
68+
</div>
5969
}
6070
</header>
71+
// script is being used here, so it doesn't render twice on desktops and break shit.
72+
<script src="/static/js/search.js" lang="javascript">
73+
</script>
6174
}

components/ui/search/search.templ

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,5 @@ templ Search() {
3434
id="search-suggestions-container"
3535
></div>
3636
</div>
37-
//
38-
<script src="/static/js/search.js" lang="javascript">
39-
</script>
37+
// script is in ui/header/header.templ
4038
}

0 commit comments

Comments
 (0)