feat: display snap indicator at the respective corner

letsfindaway commented 1 week ago

This PR addresses comment https://github.com/OpenBoard-org/OpenBoard/pull/1013#issuecomment-2360432207. The snap indicator is now shown at the point which caused the snapping. If this is outside of the visible area of the view, then the indicator is shown at the edge of the visible area.

display snap indicator at the corner of the moved object which snapped
limit indicator position by visible area of the view
move appearing of the snap indicator to the snap(QRectF) function
for rotated objects this makes it obvious that the corners of the bounding rectangle snap, not the corners of the object itself

Additionally this PR solves the problem with glitches when moving objects in a second commit.

the already known problem with glitches when moving objects and the thumbnail palette is visible gets worse when moving objects with snapping
this commit solves this problem by @Vekhir proposal to switch the view update mode depending on the current tool

letsfindaway commented 1 week ago

If you want I can of course split this PR into two, because the two commits are not very much related.

kaamui commented 3 days ago

No It's OK. But I'll have to change the design again. So for now, I'll simplify the behavior and we'll see how to improve it for 1.7.3.

OpenBoard-org / OpenBoard