Skip to content

alarm: Add support for multiple alarms #2359

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

alarm: Add support for multiple alarms #2359

wants to merge 1 commit into from

Conversation

@vkareh
Copy link
Contributor

@vkareh vkareh commented Nov 3, 2025

Replace single alarm with launcher screen showing 4 different alarms. Each alarm displays time, recurrence, and enable/disable toggle in a vertical list. Tapping alarm time opens configuration screen for that specific alarm.

InfiniSim_2025-10-31_093240 InfiniSim_2025-10-31_093246

Fixes #950

@github-actions
Copy link

github-actions bot commented Nov 3, 2025 •
edited
Loading

Build size and comparison to main:

Section Size Difference
text 383916B 1488B
data 944B 0B
bss 22672B 40B

Run in InfiniEmu

@vkareh vkareh force-pushed the multiple-alarms branch 4 times, most recently from d876707 to f303517 Compare November 7, 2025 20:42
@mark9064 mark9064 added this to the 1.17.0 milestone Nov 7, 2025
@mark9064 mark9064 added the new feature This thread is about a new feature label Nov 7, 2025
@minacode
Copy link
Contributor

minacode commented Nov 8, 2025

This is a very beautiful UI! 🙂

@tituscmd
Copy link
Contributor

tituscmd commented Nov 8, 2025

This is a very beautiful UI! 🙂

Absolutely agree!

minacode
minacode reviewed Nov 8, 2025
View reviewed changes
@vkareh vkareh force-pushed the multiple-alarms branch 4 times, most recently from 308ca10 to 561ab23 Compare November 18, 2025 21:45
@Itai-Nelken
Copy link
Contributor

Itai-Nelken commented Nov 25, 2025

I've been using this for the past few days and it's great so far, thanks for implementing this!

One bug I noticed is that pressing the red "stop ringing" button doesn't make the ringing screen go away. Pressing the button does bring you back to the alarms screen.

Something else is that the alarms screen always shows time in 24H format instead of following the setting.

@vkareh
alarm: Add support for multiple alarms
a897632 
Replace single alarm with launcher screen showing 4 different alarms.
Each alarm displays time, recurrence, and enable/disable toggle in
a vertical list. Tapping alarm time opens configuration screen for that
specific alarm.
@vkareh
Copy link
Contributor Author

vkareh commented Nov 26, 2025

@Itai-Nelken

One bug I noticed is that pressing the red "stop ringing" button doesn't make the ringing screen go away. Pressing the button does bring you back to the alarms screen.

Something else is that the alarms screen always shows time in 24H format instead of following the setting.

Fixed both issues. Thanks for calling those out.

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

Reviewers

@minacode minacode minacode left review comments

Assignees

No one assigned

Labels

new feature This thread is about a new feature

Projects

None yet

Milestone

1.17.0

Development

Successfully merging this pull request may close these issues.

Multiple Alarms

5 participants

@vkareh @minacode @tituscmd @Itai-Nelken @mark9064