After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 578109 - dragging of rotated monitors snaps wrongly
dragging of rotated monitors snaps wrongly
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: Display
2.26.x
Other Linux
: Normal normal
: ---
Assigned To: Federico Mena Quintero
Control-Center Maintainers
Depends on:
Blocks: randr-tracker
 
 
Reported: 2009-04-06 07:49 UTC by Sebastien Bacher
Modified: 2009-08-20 22:09 UTC
See Also:
GNOME target: ---
GNOME version: 2.25/2.26

Attachments
Patch to fix the reported bug (812 bytes, patch)
2009-04-09 13:09 UTC, Cristian KLEIN 		none Details | Review
don't need to figure out the angle either, then... (1.48 KB, patch)
2009-08-20 18:42 UTC, Matthias Clasen 		committed Details | Review

Description Sebastien Bacher 2009-04-06 07:49:13 UTC 
the bug has been opened on https://bugs.launchpad.net/bugs/355892

"You can drag monitors around in gnome-display-properties, and their edges snap together nicely. However if you rotate one of the monitors left (or right), it snaps at the wrong places (the snapping code doesn't swap the width and the height), leaving either a space between the monitors, or making them overlap.

I made a screencast with Istanbul, but since Istanbul's frame rate is measured in seconds per frame rather than the other way around, that screencast is utterly useless. Then I made a screencast with Byzanz, which, hopefully, will work better.

gnome-control-center 1:2.26.0-0ubuntu1
http://launchpadlibrarian.net/24846470/wtf.gif
#  screencast  (601.4 KiB, image/gif) "
Comment 1 Cristian KLEIN 2009-04-09 13:09:30 UTC 
Created attachment 132407 [details] [review]
Patch to fix the reported bug

This patch will rotate height and width in the get_geometry() function, so that all edge snapping, virtual size verifications etc. work correctly. Rotating the display using cairo_rotate() is no longer necessary.
Comment 2 Matthias Clasen 2009-08-20 18:36:31 UTC 
The patch seems to work ok for me.
Comment 3 Matthias Clasen 2009-08-20 18:42:05 UTC 
Created attachment 141273 [details] [review]
don't need to figure out the angle either, then...