ScreenMazer icon indicating copy to clipboard operation
ScreenMazer copied to clipboard

Options crashes when trying to update maze size

Open JackDarnell opened this issue 1 year ago • 7 comments

After options crashes it doesn't show that options are available for the screensaver, even after reboot.

JackDarnell avatar Jul 12 '23 03:07 JackDarnell

The same happens to me

Ninzero avatar Aug 23 '23 14:08 Ninzero

@JackDarnell Don't know if you still need this. But I have found the solution to this. If you delete

$HOME/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver.x86-64/Data/Library/Preferences/ByHost/com.alexbeals.ScreenMazer.*.plist

and then reopen the Preference, you will find the option available. As long as you do not increase maze size it should work just fine.

Ninzero avatar Aug 24 '23 08:08 Ninzero

@Ninzero Just tried it out and it works, thanks for figuring that out!!

JackDarnell avatar Aug 24 '23 14:08 JackDarnell

I bet the problem has to do with the size of the screensaver being bigger than the box it should be in in the setting screen, rather than an issue with saving the settings. It also crashes when the clock size is too big, and by looking at the preview window it seems the clock size for the preview would be larger than the actual preview size.

JackDarnell avatar Aug 25 '23 22:08 JackDarnell

Yea that has to be it, the maze size can be made smaller but when it's made larger it crashes.

JackDarnell avatar Aug 25 '23 22:08 JackDarnell

@Ninzero fixed the bug by setting the square size to 1 in the is preview section, here is the code starting at line 58 in MazeScene:

if (isPreview) { squareSize = 1 }

Going to open a PR with the update. Thanks for your help in deleting the plist file, I wanted to figure this out but the settings would always carry over to whatever new code I added lol.

JackDarnell avatar Aug 25 '23 22:08 JackDarnell

That's great. Glad I could help.

Ninzero avatar Aug 27 '23 08:08 Ninzero