InfiniTime icon indicating copy to clipboard operation
InfiniTime copied to clipboard

Alarm: Add button to set alarm time to current time

Open tituscmd opened this issue 6 months ago • 9 comments

Hey all! :)

This PR is a QoL addition to the Alarm screen. I've added a button to the screen that, when pressed, sets the alarm time to the current time. image

Tell me what you think! :smiley:

tituscmd avatar Jun 30 '25 14:06 tituscmd

Build size and comparison to main:

Section Size Difference
text 379428B 224B
data 944B 0B
bss 22536B 0B

Run in InfiniEmu

github-actions[bot] avatar Jun 30 '25 14:06 github-actions[bot]

I'm interested, what's your use case?

mark9064 avatar Jul 03 '25 12:07 mark9064

Say it's 14:37 (time of writing this) and I want to set an alarm for roughly 2hrs from now. Then I go into the alarm, press my new button and just up the hour slider by 2. And then if I want it to be a "clean" number like 16:30, I can easily do that as well.

tituscmd avatar Jul 03 '25 12:07 tituscmd

Ahhh gotcha. I think that functionality should be provided by the timer app really. It should be improved to allow hours, and #1971 solves the problem of the timer end being too subtle

mark9064 avatar Jul 03 '25 13:07 mark9064

Would this work for you? Or is the timer app not suitable for this?

mark9064 avatar Jul 04 '25 20:07 mark9064

Yeah, I think that works! I agree that the timer app with the added hours makes more sense for this use case

tituscmd avatar Jul 04 '25 21:07 tituscmd

I'll just keep this PR open tho, in case anyone wants to use it :)

tituscmd avatar Jul 05 '25 15:07 tituscmd

It definitely makes more sense to use the timer app, but I use the alarm for timers because timers don't ring, which makes them pretty useless.

I wouldn't necessarily merge this, but until timer ringing is merged, it's a nice addition

liamcharger avatar Jul 13 '25 18:07 liamcharger

Timer ringing is queued for 1.16 so hopefully that should be resolved soon

mark9064 avatar Jul 13 '25 21:07 mark9064