Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions .Jules/palette.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,3 +5,7 @@
## 2024-06-21 - Added skip-to-content link
**Learning:** Found a missing skip-to-content link, which is a key accessibility feature to help keyboard and screen reader users bypass navigation. Additionally learned that giving `<main>` `tabindex="-1"` and removing its outline when `:focus-visible` ensures proper focus handling after clicking the skip link without disruptive visual outlines.
**Action:** Always include a skip-to-content link near the start of the `body` and manage target focus appropriately.

## 2026-06-25 - Fix Header Overlap
**Learning:** When using a sticky header, clicking anchor links can cause the target element to scroll under the header, hindering the user experience.
**Action:** Use `scroll-padding-top` on the `html` element with the height of the sticky header to ensure anchor links scroll to a position just below the header.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
node_modules
venv/
4 changes: 3 additions & 1 deletion styles.css
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
--line: #d6dad8;
--white: #ffffff;
--shadow: 0 24px 70px rgba(22, 38, 61, 0.12);
--header-offset: 81px;
}

* {
Expand All @@ -23,6 +24,7 @@

html {
scroll-behavior: smooth;
scroll-padding-top: var(--header-offset);
}
Comment thread
seonghobae marked this conversation as resolved.

body {
Expand Down Expand Up @@ -113,7 +115,7 @@ a {
}

.hero {
min-height: min(620px, calc(100vh - 81px));
min-height: min(620px, calc(100vh - var(--header-offset)));
display: grid;
grid-template-columns: minmax(0, 1.15fr) minmax(340px, 0.85fr);
gap: clamp(32px, 5vw, 72px);
Expand Down