Skip to content

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

Open
tituscmd wants to merge 12 commits into
InfiniTimeOrg:mainfrom
InfiniBros:alarm_setcurrenttime
Open

Alarm: Add button to set alarm time to current time#2333
tituscmd wants to merge 12 commits into
InfiniTimeOrg:mainfrom
InfiniBros:alarm_setcurrenttime

Conversation

@tituscmd

Copy link
Copy Markdown
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! 😃

@github-actions

github-actions Bot commented Jun 30, 2025

Copy link
Copy Markdown

Build size and comparison to main:

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

Run in InfiniEmu

@mark9064

mark9064 commented Jul 3, 2025

Copy link
Copy Markdown
Member

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

tituscmd commented Jul 3, 2025

Copy link
Copy Markdown
Contributor Author

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

mark9064 commented Jul 3, 2025

Copy link
Copy Markdown
Member

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

mark9064 commented Jul 4, 2025

Copy link
Copy Markdown
Member

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

@tituscmd

tituscmd commented Jul 4, 2025

Copy link
Copy Markdown
Contributor Author

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

@tituscmd

tituscmd commented Jul 5, 2025

Copy link
Copy Markdown
Contributor Author

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

@liamcharger

liamcharger commented Jul 13, 2025

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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