Skip to content

chore: scroll to top#70

Closed
iiio2 wants to merge 1 commit intonpmx-dev:mainfrom
iiio2:chore/scroll-to-top
Closed

chore: scroll to top#70
iiio2 wants to merge 1 commit intonpmx-dev:mainfrom
iiio2:chore/scroll-to-top

Conversation

@iiio2
Copy link
Copy Markdown
Contributor

@iiio2 iiio2 commented Jan 25, 2026

resolves #59

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 25, 2026

@iiio2 is attempting to deploy a commit to the danielroe Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
npmx.dev Ready Ready Preview, Comment Jan 25, 2026 3:04pm

Request Review

@rshigg
Copy link
Copy Markdown
Contributor

rshigg commented Jan 25, 2026

I think it would be good to use CSS scrollable queries for this with javascript as a fallback. That way once the feature becomes baseline we can get rid of the js entirely.

@danielroe
Copy link
Copy Markdown
Member

this and also #71 overlap with #69 (which came first by just a few minutes!) so I'm merging it in with credit to you 🙏

also added support for scrollable queries if there's browser support - but would welcome a review from @rshigg if you want 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

scroll to top button

3 participants