Do continous scrolling when there is high resolution scroll
-
@fwsmit "Do continous scrolling when there is high resolution scroll", where is the "when there is high resolution scroll" in this commit?
-
Good question. Previously the
angleDelta().y()
value could be accumulated incontrolWheelAccumulatedDelta
and everyQWheelEvent::DefaultDeltasPerStep
(150) ticks there would be a zoom step. Now it uses the raw delta value. On touchpads or other high resolution scroll devices this gives smaller steps than 150 (maybe just steps of 1?) -
🐬 @davidhurkaDeveloperSo it means stepped scrolling is handled in steps of “150”, but it no longer aligns to the default zoom steps but just zooms by “150”? I think using the default zoom steps was somewhat sensible.
-
For mouse, users, yes. It now zooms by 150. For touchpad users it zooms in smaller steps. It does seem that I've gotten rid of the default zoom steps, like zoom to fit, etc.
I think using the default zoom steps was somewhat sensible. This is not the best place to discuss, but I think each zoom step should be roughly similar in zoom factor. It might be reasonable to do some kind of rounding to get nice zoom% numbers in the interface.
Again, it's better to discuss this on the MR this came from or somewhere in bugzilla.