Skip to content

Make polar day and polar night wallpapers more accurate#545

Closed
lulunac27a wants to merge 6 commits intot1m0thyj:mainfrom
lulunac27a:main
Closed

Make polar day and polar night wallpapers more accurate#545
lulunac27a wants to merge 6 commits intot1m0thyj:mainfrom
lulunac27a:main

Conversation

@lulunac27a
Copy link
Copy Markdown

The brightest wallpaper for polar day and polar night is at solar noon (approximately noon or 1pm local time, which is the highest point of the sun)

@t1m0thyj
Copy link
Copy Markdown
Owner

@lulunac27a Thanks for your contribution and bringing this issue to my attention! The changes made in this PR seem to only affect the theme preview dialog. There is also some code in SolarScheduler.cs that would need to be changed for the wallpaper scheduler to respect solar noon:

if (data.polarPeriod != PolarPeriod.None)
{
imageList = e.currentTheme?.dayImageList;
if (data.polarPeriod == PolarPeriod.PolarNight || JsonConfig.settings.darkMode)
{
imageList = e.currentTheme?.nightImageList;
}
segmentStart = dateNow.Date;
segmentEnd = dateNow.Date.AddDays(1);
}

I've changed this in 06825b3 and fb65795 - tried to make wallpapers more accurate for polar day and night, and added some unit tests to verify solar noon is being handled correctly. Could you download a test build from here and let me know if it works as expected? (EXE is portable version, MSIX is Store app)
https://github.com/t1m0thyj/WinDynamicDesktop/actions/runs/7454608043

@t1m0thyj
Copy link
Copy Markdown
Owner

This behavior should now be fixed in WinDD 5.4.1 to show the brightest wallpaper at solar noon.

@t1m0thyj t1m0thyj closed this Feb 16, 2024
@t1m0thyj t1m0thyj added this to the Next Release milestone Mar 17, 2024
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

Successfully merging this pull request may close these issues.

2 participants