internal/ui: bug fix: a window was shown unexpectedly at start-up

Closes #2951
This commit is contained in:
Hajime Hoshi 2024-04-07 02:52:16 +09:00
parent 5701206229
commit 9dc103491a

View File

@ -1028,12 +1028,8 @@ func (u *UserInterface) initOnMainThread(options *RunOptions) error {
} }
// Window is shown after the first buffer swap (#2725). // Window is shown after the first buffer swap (#2725).
// On Linux or UNIX, there is a problematic desktop environment like i3wm if err := glfw.WindowHint(glfw.Visible, glfw.False); err != nil {
// where an invisible window size cannot be initialized correctly (#2951). return err
if runtime.GOOS == "darwin" || runtime.GOOS == "windows" {
if err := glfw.WindowHint(glfw.Visible, glfw.False); err != nil {
return err
}
} }
if err := glfw.WindowHintString(glfw.X11ClassName, options.X11ClassName); err != nil { if err := glfw.WindowHintString(glfw.X11ClassName, options.X11ClassName); err != nil {
@ -1298,6 +1294,40 @@ func (u *UserInterface) update() (float64, float64, error) {
if err = u.window.Focus(); err != nil { if err = u.window.Focus(); err != nil {
return return
} }
if runtime.GOOS == "darwin" || runtime.GOOS == "windows" {
return
}
// On Linux or UNIX, there is a problematic desktop environment like i3wm
// where an invisible window size cannot be initialized correctly (#2951).
// Call SetSize explicitly after the window becomes visible.
fullscreen, e := u.isFullscreen()
if e != nil {
err = e
return
}
if fullscreen {
return
}
m, e := u.currentMonitor()
if e != nil {
err = e
return
}
s := m.DeviceScaleFactor()
newW := int(dipToGLFWPixel(float64(u.origWindowWidthInDIP), s))
newH := int(dipToGLFWPixel(float64(u.origWindowHeightInDIP), s))
// Even though a framebuffer callback is not called, waitForFramebufferSizeCallback returns by timeout,
// so it is safe to use this.
if err = u.waitForFramebufferSizeCallback(u.window, func() error {
return u.window.SetSize(newW, newH)
}); err != nil {
return
}
}) })
if err != nil { if err != nil {
return 0, 0, err return 0, 0, err