Skip to content

v1.1.4#110

Merged
devakesu merged 4 commits intomainfrom
1.1.4
Mar 6, 2026
Merged

v1.1.4#110
devakesu merged 4 commits intomainfrom
1.1.4

Conversation

@devakesu
Copy link
Owner

@devakesu devakesu commented Mar 6, 2026

fix(scroll): normalize wheel delta for Firefox Linux snap scrolling

  • convert wheel delta from line/page modes to pixel units before snap logic
  • use normalized delta for direction and accumulator checks
  • extract snap threshold into WHEEL_SNAP_THRESHOLD_PX constant
  • bump version to 1.1.4 in package metadata and docs

- convert wheel delta from line/page modes to pixel units before snap logic
- use normalized delta for direction and accumulator checks
- extract snap threshold into WHEEL_SNAP_THRESHOLD_PX constant
- bump version to 1.1.4 in package metadata and docs
Copilot AI review requested due to automatic review settings March 6, 2026 07:14
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR releases v1.1.4 and fixes snap-scrolling behavior by normalizing wheel delta units (notably for Firefox on Linux) so the snap accumulator/threshold logic operates consistently in pixel units.

Changes:

  • Normalize WheelEvent.deltaY from line/page delta modes into pixels before applying snap logic.
  • Use normalized deltas for direction detection and wheel-accumulator threshold checks; extract threshold into WHEEL_SNAP_THRESHOLD_PX.
  • Bump version references to 1.1.4 across package metadata and docs.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
package.json Bumps app version to 1.1.4.
package-lock.json Updates lockfile version fields to 1.1.4.
app/page.tsx Adds wheel delta normalization and uses a named snap threshold constant.
SECURITY.md Updates documented version to 1.1.4.
README.md Updates badge/version footer to 1.1.4.

💡 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>
Copilot AI review requested due to automatic review settings March 6, 2026 07:19
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.


💡 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>
Copilot AI review requested due to automatic review settings March 6, 2026 07:22
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.


💡 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>
Copilot AI review requested due to automatic review settings March 6, 2026 07:27
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 5 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.

@devakesu devakesu merged commit fd61ee6 into main Mar 6, 2026
12 checks passed
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.

3 participants