uidriver/glfw: Bug fix: Need to call updateVsync after setWindowSize

Fixes hajimehoshi#1363
momodezhou · Sep 22, 2020 · 7386fa3 · 7386fa3
1 parent 26768aa
commit 7386fa3
Showing 1 changed file with 11 additions and 7 deletions.
diff --git a/internal/uidriver/glfw/ui.go b/internal/uidriver/glfw/ui.go
@@ -831,6 +831,17 @@ func (u *UserInterface) update() error {
 		u.setInitFullscreen(false)
 	}
 
+	// Initialize vsync after SetMonitor is called. See the comment in updateVsync.
+	// Calling this inside setWindowSize didn't work (#1363).
+	_ = u.t.Call(func() error {
+		if !u.vsyncInited {
+			u.vsync = u.isInitVsyncEnabled()
+			u.updateVsync()
+			u.vsyncInited = true
+		}
+		return nil
+	})
+
 	// This call is needed for initialization.
 	u.updateSize()
 
@@ -1046,13 +1057,6 @@ func (u *UserInterface) setWindowSize(width, height int, fullscreen bool) {
 		u.windowWidth = width
 		u.windowHeight = height
 
-		if !u.vsyncInited {
-			// Initialize vsync after SetMonitor is called. See the comment in updateVsync.
-			u.vsync = u.isInitVsyncEnabled()
-			u.updateVsync()
-			u.vsyncInited = true
-		}
-
 		u.toChangeSize = true
 		return nil
 	})