Skip to content

Handle fractional macOS horizontal scroll deltas#198

Open
liuxuAP369 wants to merge 1 commit into
TomBadash:masterfrom
liuxuAP369:fix-macos-fractional-hscroll-threshold
Open

Handle fractional macOS horizontal scroll deltas#198
liuxuAP369 wants to merge 1 commit into
TomBadash:masterfrom
liuxuAP369:fix-macos-fractional-hscroll-threshold

Conversation

@liuxuAP369

Copy link
Copy Markdown

macOS can report Logitech horizontal tilt wheel movement as small fractional deltas. With the default threshold at 1, devices like the M720 require many tilt events before a mapped horizontal-scroll action fires, making desktop/browser mappings appear unresponsive. Lower the default threshold to the existing minimum effective value and allow saved integer thresholds to remain compatible with the float default.

Constraint: Observed M720 macOS delta is about 0.100006 per tilt event

Rejected: Device-specific M720 special case | other Logitech devices can emit the same fractional macOS wheel deltas

Confidence: high

Scope-risk: narrow

Tested: python -m unittest tests.test_config tests.test_engine

Not-tested: Full app GUI regression

macOS can report Logitech horizontal tilt wheel movement as small fractional deltas. With the default threshold at 1, devices like the M720 require many tilt events before a mapped horizontal-scroll action fires, making desktop/browser mappings appear unresponsive. Lower the default threshold to the existing minimum effective value and allow saved integer thresholds to remain compatible with the float default.

Constraint: Observed M720 macOS delta is about 0.100006 per tilt event

Rejected: Device-specific M720 special case | other Logitech devices can emit the same fractional macOS wheel deltas

Confidence: high

Scope-risk: narrow

Tested: python -m unittest tests.test_config tests.test_engine

Not-tested: Full app GUI regression
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.

1 participant