Skip to content

Alarm: Add button to set alarm time to current time #2333

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

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