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

Pixel scrolling reverses direction when stopping #476

Open
diocletiann opened this issue May 30, 2023 · 5 comments
Open

Pixel scrolling reverses direction when stopping #476

diocletiann opened this issue May 30, 2023 · 5 comments

Comments

@diocletiann
Copy link

diocletiann commented May 30, 2023

Thanks all the work on this, it's my favorite Neovim GUI for touchpad pixel scrolling on MacOS. Would it be possible to disable the direction reversal when scrolling stops in the middle of a row and continue moving in the direction of the scroll until the partial row is fully visible? The direction reversal is very distracting and I think this alternative could look very smooth.

@akiyosi
Copy link
Owner

akiyosi commented Jun 2, 2023

@restfuladi
Hi, Thanks for this issue report.
I have made a trial fix for this problem.

https://github.com/akiyosi/goneovim/actions/runs/5145975259

I would appreciate your feedback on whether this is in line with your expected behavior.

@akiyosi
Copy link
Owner

akiyosi commented Jun 8, 2023

@restfuladi Gentle Ping

@diocletiann
Copy link
Author

diocletiann commented Jun 9, 2023

Hey, thanks for jumping on this so fast. I tested them side by side and now I'm not sure which one I prefer, they are both jumpy when the caret hits the window edge. If you could scroll partial lines per pixel and not allow the caret into them it would be ideal. I also noticed these are both running in Intel emulation mode and I'm not sure how much that costs.

@akiyosi
Copy link
Owner

akiyosi commented Jun 10, 2023

@restfuladi
Thanks for reply.
Your opinion is well understood.
One possible approach would be, for example, to leave the amount of pixel scrolling that is less than the height of the row after a touchpad scrolling operation. However, this would require manually adjusting back to the exact row-by-row viewport after pixel scrolling, which some people may not like.

@akiyosi
Copy link
Owner

akiyosi commented Jun 10, 2023

I also noticed these are both running in Intel emulation mode and I'm not sure how much that costs.

goneovim does not yet prepare binaries for the Apple M1 arm64. This will be a topic for future improvement.

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

2 participants