• Status Closed
  • Percent Complete
  • Task Type Patches
  • Category Applications
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by xaviergr - 2007-09-23
Last edited by kugel. - 2011-06-05

FS#7819 - Unify alarm and time/date screen

This patch is a work in progress.

It will remove alarm_menu.c and alarm_menu.h from the source and use the existing code of the time/date screen to set the alarm.

This move has three advantages:
1) Reduces binary size.
2) Makes the alarm screen consistent with the time/date screen.
3) Gives more control on setting the alarm.

This patch will have even more meaning when the RTC alarm drivers are modified to accept months, years etc. as a trigger.
Currently all other entries except hours and minutes will not affect the alarm usage.

Closed by  kugel.
2011-06-05 11:32
Reason for closing:  Out of Date
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

There doesn't seem to be interest in this anymore. Open a new task if this is wrong.

I quite like the ease and effectiveness of the original alarm setting screen. How many more key presses do you need to set the alarm with the new method? Why on earth would you want to use month or even years as an alarm trigger on a DAP?

roolku: on the contrary, fewer key presses, more control and you don’t have to think about the date if you don’t want to set it.
There is nothing effective to have to pass 5 hours on an interval of 5 minutes to set your preferred time. The time/date screen allows you to set the time easier, faster and more accurately.


Available keyboard shortcuts


Task Details

Task Editing