-
Notifications
You must be signed in to change notification settings - Fork 136
Description
Repro is with Zui commit ca047c0. I bumped into this issue while going through the test permutations on all platforms before merging #3084.
The first repro video is taken with Zui Insiders release 1.7.1-12 which is based on Zui commit ca047c0. As we can see, the app initially launches ok, but then when I double-click the title bar to maximize the window, the app window disappears due to a crash.
Repro.mp4
The second repro video is taken by starting Zui in Dev Mode at commit ca047c0. This more clearly shows the root cause of the crash being an Electron SIGSEGV.
Repro-Dev-Mode.mp4
That Zui commit ca047c0 happens to be related to #3061 where Zui's Electron dependency was advanced from 28.0.0 to 30.0.1. In follow-on Dev Mode tests I advanced the Electron dependency one patch release at a time and found the problem went away at Electron 30.0.5. Sure enough, there's an Electron issue electron/electron#41839 that describes this exact symptom as having been fixed in Electron release v30.0.5, so we'll want to advance Zui's Electron dependency to a release like this one that includes the fix.
(Side note: I initially found this issue on Rocky Linux (RPM installer) after not having seen it on Ubuntu Linux (DEB installer) and therefore thought incorrectly at first that it was installer-related. Of course, in hindsight I can see the problem has to do with saved app state, since coincidentally on my Ubuntu VM I happened to have previously had Zui launched in a non-maximized state and didn't happen to have a reason to maximize it, so I didn't happen to bump into this bug in my testing there. By comparison, on my Rocky Linux VM I don't think I'd previously done any testing with Zui Insiders, so upon first launch the app window happened to be maximized, triggering the bug.)