ultimate_alarm_clock icon indicating copy to clipboard operation
ultimate_alarm_clock copied to clipboard

Bug: Incrementing the time from snooze does not portray the time left in a user friendly way

Open Wreck-X opened this issue 1 year ago • 4 comments

Description

When incremented enough, The timer is always of the format MM:SS, I believe its better to change the format of the time when it hits the 60 minutes mark to either HH:MM:SS or switch to HH:MM.

additionally i also think it would be better to move the incrementing buttons towards the bottom that is above the snooze button making it easier for the users to tap on them.

I would like to be assigned to this issue

Screenshots

image

Wreck-X avatar Mar 11 '24 19:03 Wreck-X

if its approved please assign this to me as id like to work on this :)

Wreck-X avatar Mar 11 '24 19:03 Wreck-X

Hi @Wreck-X, I was the one adding snooze buttons, I specifically positioned them close to the timer display. If it were closer to the 'dismiss' button, in a user workflow it is possible to hit the 'dismiss' button instead of the add snooze timers, and failing the whole point of the alarm clock app. And for the display of hours in timer, I wonder if that's necessary because then we would have the '00' sitting there (if implemented according to the current design) idle most of the time, as a user would rarely go past 30 mins of snoozing, let alone an hour. However, if the maintainers think that the changes should be made, either of us can work on it.

mohnishdeshpande avatar Mar 11 '24 23:03 mohnishdeshpande

@mohnishdeshpande The timer need not necessarily display the 00: all the time it could add the field incase it ever does go over the 60 minute mark imo, as for the position of the buttons I guess it's fine to keep it at the top if the idea was to avoid misclicks.

Wreck-X avatar Mar 12 '24 02:03 Wreck-X

Although I doubt anyone is going to be snoozing for over 60 minutes, I think its good to handle the case anyway. I'll assign this to you, ler's show the HH only when it is necessary. As for the buttons - I agree with @mohnishdeshpande too close to dismiss would be a bad idea.

MarkisDev avatar Mar 26 '24 14:03 MarkisDev

Hi @Wreck-X are you still working on this issue If not, could you assign this issue to me @MarkisDev @AryanSarafDev

ganidande905 avatar Dec 10 '24 15:12 ganidande905

im not working on this anymore feel free to work on it

Wreck-X avatar Dec 10 '24 15:12 Wreck-X

Hi @Wreck-X are you still working on this issue If not, could you assign this issue to me @MarkisDev @AryanSarafDev

@ganidande905 I think this issue is not required anymore because the ui and flow has shifted to a different implementation. But you can think of implementing a snooze button for the new timer ui for individual timers and create a new issue for it. I am closing this issue. You can also try implementing quick timer options for like 1 min 5 min etc!

AryanSarafDev avatar Dec 10 '24 16:12 AryanSarafDev