Conversation
- scope scroll-behavior/overscroll-behavior-y to @media (pointer: fine) - add touch device guard to reactive-glow.js scroll listener - bump version to 1.1.3 across package.json, README.md, SECURITY.md - update react-icons 5.5.0 → 5.6.0 - update GitHub Actions: harden-runner v2.15.1, setup-node v6.3.0, trivy-action v0.34.2, setup-buildx-action v4.0.0, login-action v4.0.0, build-push-action v7.0.0, sbom-action v0.23.0, upload-artifact v7.0.0, attest-build-provenance v4.1.0 Co-authored-by: devakesu <61821107+devakesu@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix scroll performance issues and update version references
fix(mobile): scope desktop scroll CSS to pointer:fine, guard reactive-glow scroll listener — v1.1.3
Mar 6, 2026
Copilot
AI
changed the title
fix(mobile): scope desktop scroll CSS to pointer:fine, guard reactive-glow scroll listener — v1.1.3
fix(mobile): scope desktop scroll CSS to pointer:fine + guard reactive-glow scroll listener — v1.1.3
Mar 6, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR reduces mobile scroll jank by preventing desktop-only scroll CSS and scroll-driven glow effects from running on touch devices, and bumps the release to v1.1.3 with dependency/CI action updates.
Changes:
- Scope
scroll-behavior,overscroll-behavior-y, andscroll-snap-typeto@media (pointer: fine)to avoid interfering with touch scrolling. - Skip
reactive-glow.jsscroll listener work on(pointer: coarse)devices to reduce scroll-frame work on mobile. - Bump version to
1.1.3, updatereact-icons, and refresh pinned GitHub Actions.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
app/globals.css |
Moves desktop-oriented scroll CSS under a pointer:fine media query to preserve native touch scrolling behavior. |
public/js/reactive-glow.js |
Adds a touch-device guard to avoid scroll rAF + DOM mutations on mobile. |
package.json |
Version bump to 1.1.3 and updates react-icons to 5.6.0. |
package-lock.json |
Locks updated dependency tree/version and aligns Node engine metadata. |
README.md |
Updates displayed version to 1.1.3. |
SECURITY.md |
Updates documented version to 1.1.3. |
.github/workflows/deploy.yml |
Updates pinned action SHAs/versions used in CI/CD pipeline. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Devanarayanan <fusion@devakesu.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Devanarayanan <fusion@devakesu.com>
Merged
devakesu
added a commit
that referenced
this pull request
Mar 6, 2026
* fix(mobile): scope desktop scroll CSS to pointer:fine + guard reactive-glow scroll listener — v1.1.3 (#106) * Initial plan * fix: mobile scroll lag + bump to v1.1.3 - scope scroll-behavior/overscroll-behavior-y to @media (pointer: fine) - add touch device guard to reactive-glow.js scroll listener - bump version to 1.1.3 across package.json, README.md, SECURITY.md - update react-icons 5.5.0 → 5.6.0 - update GitHub Actions: harden-runner v2.15.1, setup-node v6.3.0, trivy-action v0.34.2, setup-buildx-action v4.0.0, login-action v4.0.0, build-push-action v7.0.0, sbom-action v0.23.0, upload-artifact v7.0.0, attest-build-provenance v4.1.0 Co-authored-by: devakesu <61821107+devakesu@users.noreply.github.com> * Update .github/workflows/deploy.yml Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Devanarayanan <fusion@devakesu.com> * Update app/globals.css Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Devanarayanan <fusion@devakesu.com> --------- Signed-off-by: Devanarayanan <fusion@devakesu.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: devakesu <61821107+devakesu@users.noreply.github.com> Co-authored-by: Devanarayanan <fusion@devakesu.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * fix: tighten hybrid-device touch guards and pin Node version in CI (#108) * Initial plan * fix: apply review feedback — any-pointer guard, hybrid CSS fix, pin Node 20.19.0 Co-authored-by: devakesu <61821107+devakesu@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: devakesu <61821107+devakesu@users.noreply.github.com> * fix(css): replace MQ4 negation with MQ3-compatible scroll behavior guards (#109) * Initial plan * fix(css): replace MQ Level-4 negation with MQ3-compatible two-block approach for scroll behavior Co-authored-by: devakesu <61821107+devakesu@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: devakesu <61821107+devakesu@users.noreply.github.com> --------- Signed-off-by: Devanarayanan <fusion@devakesu.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Three unconditional behaviours were causing mobile scroll lag:
overscroll-behavior-y: containkills iOS rubber-band/momentum,scroll-behavior: smoothglobally fights native momentum scroll, and thereactive-glow.jsscroll listener was firing rAF + DOM mutations on every mobile scroll frame.CSS —
app/globals.cssscroll-behavior: smooth,overscroll-behavior-y: contain, andscroll-snap-type: noneunder@media (pointer: fine)so they never apply on touch devicesJS —
public/js/reactive-glow.js(pointer: coarse)guard around the scroll event listener; touch devices skip the rAF paint work entirely, matching the existing pattern inparallax.jsVersion bump & dependency updates
1.1.2 → 1.1.3acrosspackage.json,package-lock.json,README.md,SECURITY.mdreact-icons5.5.0 → 5.6.0harden-runnerv2.15.1,setup-nodev6.3.0,trivy-actionv0.34.2,docker/setup-buildx-actionv4.0.0,docker/login-actionv4.0.0,docker/build-push-actionv7.0.0,anchore/sbom-actionv0.23.0,actions/upload-artifactv7.0.0,actions/attest-build-provenancev4.1.0✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.