diff options
| author | Sven Gothel <[email protected]> | 2013-12-19 12:54:07 +0100 |
|---|---|---|
| committer | Sven Gothel <[email protected]> | 2013-12-19 12:54:07 +0100 |
| commit | 9b2bbaf82ed410096ac279542357daefc0d6dc7f (patch) | |
| tree | f8ff91a52daf9fdaa4278a8d17a552b9e0348874 | |
| parent | 0e1b4bf1b17b10933c3ccdfea5cef7eca6aad2fb (diff) | |
Bug 924: Remove position criteria from reparent/fullscreen success, WM's only regard custom position as a hint (X11).
| -rw-r--r-- | src/newt/classes/jogamp/newt/WindowImpl.java | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/src/newt/classes/jogamp/newt/WindowImpl.java b/src/newt/classes/jogamp/newt/WindowImpl.java index 29d3b9ee2..e3174bd9f 100644 --- a/src/newt/classes/jogamp/newt/WindowImpl.java +++ b/src/newt/classes/jogamp/newt/WindowImpl.java @@ -1442,9 +1442,9 @@ public abstract class WindowImpl implements Window, NEWTEventConsumer ok = WindowImpl.this.waitForSize(width, height, false, TIMEOUT_NATIVEWINDOW); } if(ok) { - ok = WindowImpl.this.waitForPosition(true, x, y, TIMEOUT_NATIVEWINDOW); - } - if(ok) { + // Position mismatch shall not lead to reparent failure + WindowImpl.this.waitForPosition(true, x, y, TIMEOUT_NATIVEWINDOW); + requestFocusInt( 0 == parentWindowHandle /* skipFocusAction if top-level */); display.dispatchMessagesNative(); // status up2date } @@ -2158,8 +2158,9 @@ public abstract class WindowImpl implements Window, NEWTEventConsumer if(ok) { ok = WindowImpl.this.waitForSize(w, h, false, TIMEOUT_NATIVEWINDOW); } - if(ok && !fullscreen) { - ok = WindowImpl.this.waitForPosition(true, x, y, TIMEOUT_NATIVEWINDOW); + if(ok && !_fullscreen) { + // Position mismatch shall not lead to fullscreen failure + WindowImpl.this.waitForPosition(true, x, y, TIMEOUT_NATIVEWINDOW); } if(ok) { requestFocusInt(fullscreen /* skipFocusAction if fullscreen */); |