Skip to content

Conversation

@voloved
Copy link
Collaborator

@voloved voloved commented Oct 28, 2025

Location Setting Face

This PR updates the Sunrise/Sunset face. Now, instead of requiring the long/lat to set the location, you can set from the most populated cities.

If you have your timezone set outside of UTC, it'll allow you to select from a preset of cities only within that timezone.
If it's set to UTC (which is the default), it'll let you set from any city in the presets.
If no cities are found in that timezone, the user will need to manually input their long/lat the same way that they currently do.
At the end of every list of presets, there is a CUSTOM option, which allows the user to always input their own long/lat.
The city the user will see when entering the location setting is the city that shares their currently-selected long/lat. If no cities share it, it'll display CUSTOM.

The cities were chosen as the all cities with a 2025 population of at least 1,000,000 that are not within 500km of a more populated city that shares its timezone.
Since location is mainly used for sunrise data, it won't care too much if you live in Taicang, but set it to Shanghai.
The city list can be found here.

location-presets-sun.mp4

@voloved voloved marked this pull request as draft November 7, 2025 02:46
@voloved voloved marked this pull request as ready for review November 12, 2025 23:10
@voloved voloved marked this pull request as draft November 12, 2025 23:10
@voloved voloved marked this pull request as ready for review November 12, 2025 23:15
@voloved
Copy link
Collaborator Author

voloved commented Nov 22, 2025

@joeycastillo , Let me know what your thoughts are on this change to sunrise/sunset face.
It's a pretty big change for setting locations, but I think it's worth it to not need to manually set the long-lat

@tandy-1000
Copy link

tandy-1000 commented Jan 14, 2026

Great work!
This is related to my work too: separate the set location face from the sunrise sunset complication so that I could use it for my prayer times face.

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