Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Options crashes when trying to update maze size #9

Open
JackDarnell opened this issue Jul 12, 2023 · 7 comments
Open

Options crashes when trying to update maze size #9

JackDarnell opened this issue Jul 12, 2023 · 7 comments

Comments

@JackDarnell
Copy link

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

@Ninzero
Copy link

Ninzero commented Aug 23, 2023

The same happens to me

@Ninzero
Copy link

Ninzero commented Aug 24, 2023

@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.

@JackDarnell
Copy link
Author

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

@JackDarnell
Copy link
Author

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
Copy link
Author

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

@JackDarnell
Copy link
Author

@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.

@Ninzero
Copy link

Ninzero commented Aug 27, 2023

That's great. Glad I could help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants