Call set_time_face on long press of alarm. Move alert toggle to light. #456
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Since we were talking about this :)
Instead of trying to merge the functionality into every watch face that might need it, I simply call the set_time_face directly on long press of alarm.
I've moved the alert functionality onto long press of light.
Tested on the simulator.. seems to work fine. I did watch_face first then realized I was using simple_watch_face too but the changes are not large.
It would be nice if there was a formal way for faces to call complications/settings like this, but this works.