-
-
Notifications
You must be signed in to change notification settings - Fork 355
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
Mouse hover during scrolling sometimes janks and focuses the wrong window #259
Comments
A bit of update on more behaviors I've observed, and it looks like it's because of the
|
That info you shared is super helpful! It narrows down the issue. It might be a limitation from the OS, or we may be able to override the mouse-scroll event, and force it to happen whatever modifier is pressed. I can't work on this at the moment unfortunately since I don't have a mouse at hand. |
That's no problem at all! For now I can always use the arrow keys (or keep cycling tab key) if I really need to. Also holding my breath for the support of up/down arrow keys 😉 |
I found a mouse, and investigated. For me the issue only happens with the Indeed, the Going back to you saying that
I downloaded Firefox, and I get the following behavior:
How does Could you please test with my local build that fixes the issue with Update: the |
Sorry I forgot to mention that I disabled I disabled the zoom in Firefox by setting However, for now I decided to switch AltTab to use BUT... I think I found the culprit! I have this app called SteerMouse to add more functionality to my mice, and I just found out by default it set the mice to |
It's been my own applications causing the isuse, so I apologize for making you invest your time on this. I will close the issue! |
Nono actually it was very interesting to investigate the issue, and even though it was not the original issue you had, i still fixed the issue with the It was incredibly hard to find how to do it. I had to read about Responsive Scrolling, and find a way to modify the It also made me realize that the bouncing after a fast trackpad scroll creates a bouncing animation, and that animation doesn't play well with the mouse hover scrolling behavior i added a while ago to help always show mouse hover thumbnails entirely) I'll re-open the ticket to track these 2 issues. I'll close it when it's merge onto |
It's absolutely inspiring seeing your enthusiasm working on this @lwouis. It's a completely free app yet you are very active with the issue with all the excitement. I'm a programmer myself (not doing apps using XCode though), so I admire you very much! Do you happen to have a patreon/donation link? I can't give a lot but I want to give some. |
Thank you for the kind words @ngocphamm! Don't worry about donating! People like you make this enterprise worth the time investment. I find great joy thinking about people using this app, or people coming here to discuss things and having great interactions together :) |
I merged the fix for the I thought it was related to when the view bounces once it reaches the last bit of content, but it's not. It's about mouse hovering while a scroll is happening. It only happens with the trackpad, as the mouse has no inertia-scrolling. |
I'm not sure about the technical side of the behavior, but I guess some other mice can also have inertia-scrolling-like feature, like the Logitech Master MX. Please let me know if you want me to test it for you, as I do have that mouse. |
I can reproduce with the trackpad locally so reproduction is fine. The issue is understanding why it's happening. I think i'll film with my iphone in slow-mo to better understand what's happening as it happens so fast. There are a lot of subtleties with scrolling behavior for instance it took me a while to realize yesterday that if you scroll then move your mouse, the OS will slow down the previous scroll, probably because they assume you want to act now so they stop the inertia quickly |
As mentioned in #256
Tried with 2 mice: Logitech MX Master, and an Elecom trackball. Laptop's trackpad 2-finger scroll is fine though.
The text was updated successfully, but these errors were encountered: