Skip to content

yet more QRect right() fixes

Harald Sitter requested to merge work/sitter/setpositionoffbyone into master

QRect::right and topRight are off by one for historical reasons. This caused incorrect default positioning when a (new) screen gets enabled and automatically positioned. To solve this problem simply add 1.

This is the same kind of solution we've already employed for our snapping system. Unfortunately enough the relevant code paths for programatic positioning don't use the snapping system - I've add a note about this possibly leading to more consistent results.

BUG: 455394

Merge request reports