We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Adjust window location in ConfigureNotify

patch-1.diff (text/plain), 2.13 KB, created by BurnSpamAddress on 2013-10-20 15:28:05 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: BurnSpamAddress

Created: 2013-10-20 15:28:05 UTC

Size: 2.13 KB

patch

obsolete

># HG changeset patch
># User Stefanos Apostolopoulos <stapostol@gmail.com>
># Date 1382282623 -7200
>#      Sun Oct 20 17:23:43 2013 +0200
># Node ID 909b0d7fe4dd0af23585e750bd3396a0b53d2a6b
># Parent  2f2f0b3b4702482cb47a98904fb1822080e60266
>Fix bug 1300 by querying current border size in ConfigureNotify, and adjusting window coordinates accordingly.
>
>diff -r 2f2f0b3b4702 -r 909b0d7fe4dd src/video/x11/SDL_x11events.c
>--- a/src/video/x11/SDL_x11events.c	Sat Oct 19 01:29:23 2013 -0700
>+++ b/src/video/x11/SDL_x11events.c	Sun Oct 20 17:23:43 2013 +0200
>@@ -519,10 +519,32 @@
>                    xevent.xconfigure.x, xevent.xconfigure.y,
>                    xevent.xconfigure.width, xevent.xconfigure.height);
> #endif
>+            long border_left = 0;
>+            long border_right = 0;
>+            long border_top = 0;
>+            long border_bottom = 0;
>+            if (data->xwindow) {
>+                Atom _net_frame_extents = XInternAtom(display, "_NET_FRAME_EXTENTS", 0);
>+                Atom type;
>+                int format;
>+                unsigned long nitems, bytes_after;
>+                unsigned char *property;
>+                XGetWindowProperty(display, data->xwindow,
>+                    _net_frame_extents, 0, 16, 0,
>+                    XA_CARDINAL, &type, &format,
>+                    &nitems, &bytes_after, &property);
>+
>+                border_left = ((long*)property)[0];
>+                border_right = ((long*)property)[1];
>+                border_top = ((long*)property)[2];
>+                border_bottom = ((long*)property)[3];
>+            }
>+
>             if (xevent.xconfigure.x != data->last_xconfigure.x ||
>                 xevent.xconfigure.y != data->last_xconfigure.y) {
>                 SDL_SendWindowEvent(data->window, SDL_WINDOWEVENT_MOVED,
>-                                    xevent.xconfigure.x, xevent.xconfigure.y);
>+                                    xevent.xconfigure.x - border_left,
>+                                    xevent.xconfigure.y - border_top);
>             }
>             if (xevent.xconfigure.width != data->last_xconfigure.width ||
>                 xevent.xconfigure.height != data->last_xconfigure.height) {

Actions: View

Attachments on bug 1300: 697 | 1382 | 1383

We are currently migrating Bugzilla to GitHub issues.Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.