Conversation
…e-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>
There was a problem hiding this comment.
Pull request overview
This PR targets mobile scroll performance regressions by limiting desktop-oriented scroll CSS to fine-pointer environments and by disabling the reactive glow scroll pulse work on touch devices. It also bumps the project version to v1.1.3 and updates dependencies/CI actions.
Changes:
- Scope smooth scrolling / overscroll containment / scroll-snap disabling to
@media (pointer: fine)in global CSS. - Add a touch-device guard to skip
reactive-glow.jsscroll listener work on touch devices. - Bump version to
1.1.3, updatereact-iconsto5.6.0, and refresh pinned GitHub Actions.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
public/js/reactive-glow.js |
Adds a matchMedia-based guard to skip scroll-driven glow pulsing on touch devices. |
app/globals.css |
Moves scroll-behavior/overscroll behavior into a fine-pointer media query to avoid impacting touch scrolling. |
package.json |
Bumps app version and updates react-icons. |
package-lock.json |
Updates lockfile for version bump, Node engine constraint, and dependency update. |
README.md |
Updates displayed project version badge/footer version. |
SECURITY.md |
Updates documented version. |
.github/workflows/deploy.yml |
Updates pinned GitHub Actions and adjusts Node setup for CI. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot open a new pull request to apply changes based on the comments in this thread |
) * 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>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot open a new pull request to apply changes based on the comments in this thread |
…ards (#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>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 7 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
Update .github/workflows/deploy.yml
Update app/globals.css