feat: pin package header 📌 #566
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
thasmo
force-pushed
the
package-header-sticky
branch
from
7ac42c6 to
256db81
Compare
OrbisK
reviewed
Jan 31, 2026
thasmo
force-pushed
the
package-header-sticky
branch
from
256db81 to
3b0d791
Compare
thasmo
force-pushed
the
package-header-sticky
branch
from
3b0d791 to
8071aaa
Compare
OrbisK
reviewed
Jan 31, 2026
thasmo
force-pushed
the
package-header-sticky
branch
from
8071aaa to
18256fa
Compare
Contributor
Author
|
Ah, something's messed up. Going to fix this. 🤔 |
thasmo
force-pushed
the
package-header-sticky
branch
from
18256fa to
0579ad8
Compare
Contributor
Author
Should be fine now. |
Collaborator
|
Is it not possible to have a sticky header through CSS rather than manually adding a scroll listener? Maybe I'm missing something 🤔 |
Contributor
Author
It is! It's "only" needed for the bottom border, which gets added conditionally when the header is in the "pinned state". |
thasmo
force-pushed
the
package-header-sticky
branch
from
0579ad8 to
a1bb220
Compare
thasmo
force-pushed
the
package-header-sticky
branch
from
a1bb220 to
0627825
Compare
Collaborator
|
ooh yeah i understand now! thanks for clarifying. its a shame CSS doesn't have a pseudo-class for this 😅 |
thasmo
force-pushed
the
package-header-sticky
branch
from
0627825 to
5286991
Compare
OrbisK
approved these changes
Feb 1, 2026
thasmo
force-pushed
the
package-header-sticky
branch
from
5286991 to
30e5ce2
Compare
43081j
approved these changes
Feb 1, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pins the package header during scrolling.
package-header-pin.mp4
The template changes look a bit wild, but I basically just kept the now-pinned-content in the already existing
<header>element and just moved the rest into a new element below the<header>element.Eager to get feedback, especially on the
checkHeaderPositionfunction.❤️