Skip to content

Conversation

tituscmd
Copy link
Contributor

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! 😃

Copy link

github-actions bot commented Jun 30, 2025

Build size and comparison to main:

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

Run in InfiniEmu

@mark9064
Copy link
Member

mark9064 commented Jul 3, 2025

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

@mark9064 mark9064 added enhancement Enhancement to an existing app/feature UI/UX User interface/User experience labels Jul 3, 2025
@tituscmd
Copy link
Contributor Author

tituscmd commented Jul 3, 2025

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.

@mark9064
Copy link
Member

mark9064 commented Jul 3, 2025

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

mark9064 commented Jul 4, 2025

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

@tituscmd
Copy link
Contributor Author

tituscmd commented Jul 4, 2025

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

@tituscmd
Copy link
Contributor Author

tituscmd commented Jul 5, 2025

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

@liamcharger
Copy link
Contributor

liamcharger commented Jul 13, 2025

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

@mark9064
Copy link
Member

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhancement to an existing app/feature UI/UX User interface/User experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants