Skip to content
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

Alarms: show actual time #424

Closed
jtagcat opened this issue Aug 10, 2020 · 3 comments · Fixed by #428
Closed

Alarms: show actual time #424

jtagcat opened this issue Aug 10, 2020 · 3 comments · Fixed by #428
Labels
Milestone

Comments

@jtagcat
Copy link

jtagcat commented Aug 10, 2020

Right now it's 'n hours after x', would be nice to see n+x displayed.

@forrestguice
Copy link
Owner

This would be useful. Perhaps tapping this label could cause it to animate between n+x and back again. It'll be included in the next release.

@jtagcat
Copy link
Author

jtagcat commented Aug 12, 2020

This would be useful. Perhaps tapping this label could cause it to animate between n+x and back again. It'll be included in the next release.

IMO there's room for the time to be displayed without tapping.

Natural expectation would be editing, when tapping on the time, and most people might not notice that.

@forrestguice
Copy link
Owner

This detail and a bunch of others have been changed in v0.13.0. The actual time is now displayed for all non-focused items, but once focused switches to the n+x format. Also, editing alarms has been moved into its own activity where the n+x format is displayed by default (but momentarily animates/transitions to the actual time whenever those options are changed). It requires more clicks to navigate the UI, but there is a lot more room in general. I no longer feel like these features are being hidden.

It should land in F-Droid in the next week. Feedback welcome.

@forrestguice forrestguice added this to the v0.13.0 milestone Sep 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants