Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Negative scroll multiplier not applied in vim #1299

Closed
atonyngo opened this issue Jan 11, 2019 · 2 comments
Closed

Negative scroll multiplier not applied in vim #1299

atonyngo opened this issue Jan 11, 2019 · 2 comments

Comments

@atonyngo
Copy link

atonyngo commented Jan 11, 2019

As of release version 0.13.2, negative scrolling multipliers in kitty.conf aren't applied properly in vim. I have narrowed the problem down locally to this merge.

For reference I am using a touchpad on a Dell XPS 9730 on Arch Linux. The reverse multiplier seems to be applied fine in other applications such as less. I also have both wheel_scroll_multiplier and touch_scroll_multiplier in my config both set to a negative value (-3).

@Luflosi
Copy link
Contributor

Luflosi commented Jan 11, 2019

This was my pull request #1263, which fixed #1257. Sorry, evidently I didn't think about negative multipliers. Multiplying s with -1 if OPT(wheel_scroll_multiplier) is less than 0 should fix this issue. @kovidgoyal can you think of a better fix? If not, I can make a pull request.

@kovidgoyal
Copy link
Owner

@Luflosi seems ok to me, go ahead.

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

No branches or pull requests

3 participants