BetterDisplay icon indicating copy to clipboard operation
BetterDisplay copied to clipboard

Long press resolution slider favorite selection cycle might be triggered inadvertently

Open schlomo opened this issue 9 months ago • 14 comments

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?

schlomo avatar May 09 '24 13:05 schlomo

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?

waydabber avatar May 09 '24 13:05 waydabber

I have this for a while already.

schlomo avatar May 09 '24 13:05 schlomo

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?

waydabber avatar May 09 '24 13:05 waydabber

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

schlomo avatar May 09 '24 13:05 schlomo

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 avatar May 09 '24 13:05 waydabber

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.

https://github.com/waydabber/BetterDisplay/assets/101384/184a19d4-e7d2-4c4a-9ac9-d5013b2d8499

schlomo avatar May 09 '24 13:05 schlomo

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

waydabber avatar May 09 '24 13:05 waydabber

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 avatar May 09 '24 14:05 waydabber

That indeed helped, thanks a lot!

schlomo avatar May 09 '24 15:05 schlomo

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 avatar May 09 '24 15:05 waydabber

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 avatar May 15 '24 18:05 waydabber

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).

waydabber avatar May 15 '24 18:05 waydabber

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

schlomo avatar May 21 '24 08:05 schlomo

Thanks for the feedback @schlomo! :)

waydabber avatar May 21 '24 10:05 waydabber