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

Long press resolution slider favorite selection cycle might be triggered inadvertently #2954

Closed
schlomo opened this issue May 9, 2024 · 14 comments
Assignees
Labels
bug Something isn't working done All tasks are completed released Released workaround available A bug has a workaround
Milestone

Comments

@schlomo
Copy link

schlomo commented May 9, 2024

Using BetterDisplay Pro Version 2.3.3 Build 29731

image image ... and more

The Display Mode submenu is cycling like shown in this video:
https://github.com/waydabber/BetterDisplay/assets/101384/92c5d760-0041-401b-9dbc-1900578f03c8

No idea why, to be honest. Could this be a bug?

@waydabber
Copy link
Owner

Wow, this does not look good, but have no idea how this could happen or how to recreate the issue. :). Does this happen always or was it a one time occurrence?

@schlomo
Copy link
Author

schlomo commented May 9, 2024

I have this for a while already.

@waydabber
Copy link
Owner

Very interesting. i see the slider animated as well. Is that the case? Normally if you hold the mouse button over the resolution slider, you can toggle favorites. It's like as if somehow a constant mouse press is being detected by the app.

Do you use maybe some kind of third party app or accessibility setting that might have an effect like that?

@schlomo
Copy link
Author

schlomo commented May 9, 2024

No, and I don't observe unexpected mouse clicks anywhere or anytime else

@waydabber
Copy link
Owner

But do you see the resolution slider being animated and favorite numbers changing? Does this happen after you restart the app? Somehow the slider acts as if there was a constant mouse down without mouse up. Not sure why though.

@waydabber waydabber added the investigating Investigating issue label May 9, 2024
@schlomo
Copy link
Author

schlomo commented May 9, 2024

Yes, I see the resolution slider changing.

It happened after I restart BetterDisplay and toggle the Display Notch option on and off a few times.

Screen.Recording.2024-05-09.at.14.39.41.mov

@waydabber
Copy link
Owner

Hmm. All right. I'll try to recreate this issue, so far I could not, no matter how I toggle the notch. :)

@waydabber waydabber added the bug Something isn't working label May 9, 2024
@waydabber
Copy link
Owner

It will take some time for me to figure out what's wrong, I did not encounter such an issue so far. Meanwhile you can fix this by turning this off:

Screenshot 2024-05-09 at 16 52 46

Thanks for the patience with this!

@waydabber waydabber added the workaround available A bug has a workaround label May 9, 2024
@waydabber waydabber self-assigned this May 9, 2024
@schlomo
Copy link
Author

schlomo commented May 9, 2024

That indeed helped, thanks a lot!

@waydabber
Copy link
Owner

No idea what was wrong, but I now added an extra protection that the longPress activity can start only when the leftMouseDown is also happening during the valueChanged event, this should help avoid any unwanted triggering of the slider favorite setting behavior.

When possible, I'll add an updated build with this, will post the link here as well for testing.

Thank you!

@waydabber waydabber added done All tasks are completed unreleased Not released yet in beta form and removed investigating Investigating issue labels May 9, 2024
@waydabber waydabber added this to the v2.3.4 milestone May 9, 2024
@waydabber waydabber changed the title Bug: Resolution list cycling by itself Long press resolution favorite selection cycle might be triggered inadvertently May 9, 2024
@waydabber waydabber changed the title Long press resolution favorite selection cycle might be triggered inadvertently Long press resolution slider favorite selection cycle might be triggered inadvertently May 9, 2024
@waydabber
Copy link
Owner

Note - here is an interim build with this and some other bug fixes:

https://github.com/waydabber/BetterDisplay/releases/download/v2.0.0-pre-release/BetterDisplay-v2.3.4-b29903-pre.zip

@waydabber
Copy link
Owner

I'd appreciate if you could test this @schlomo and confirm if the issue is fixed (as I was not able to reproduce it, so fixed it kinda blindly by guessing what might be a problem based on the code).

@schlomo
Copy link
Author

schlomo commented May 21, 2024

Hi @waydabber thanks a lot! I installed the pre-build and couldn't observe the problem any more.

@waydabber
Copy link
Owner

Thanks for the feedback @schlomo! :)

@waydabber waydabber added released Released and removed unreleased Not released yet in beta form labels May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working done All tasks are completed released Released workaround available A bug has a workaround
Projects
None yet
Development

No branches or pull requests

2 participants