-
Notifications
You must be signed in to change notification settings - Fork 121
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
Setting alert time clock in the past. #312
Comments
I'm not 100% sure this is a bug; obviously you won't be able to do something by the due date if it's in the past, but at the same time I might have forgotten to add a task in the past and I want to add it now while maintaining the original due date, so I know how late I am. I think I'd leave it at is. Or maybe allow it but display a dialog about it in case the user is just making a mistake. |
Alright. Anyway, the due time is set to Due: - in this case which needs to be adjusted. |
I think we should display a snack bar message if the user sets a past time instead of considering it as a bug, as mentioned by @cfsmp3. |
I found a snackBar was already being used but it was not in-foucs since it used ScaffoldMessanger and we were working on the AlertDialog, so it displayed the snackBar message in the Scaffold. Also I high lighted the time, if it was now or in the past, set it to red just to get user's attention. |
Fixed in #315 |
Describe your issue
Right now when the user sets the time to be alerted for their task through the clock. You can set the time to be in the past, as in if it is currently 2PM. The alert time can be set to 1PM.
Steps to reproduce
None
What was the expected result?
To only be able to set the alert time in the future.
Put here any screenshots or videos (optional)
Record_2024-02-22-03-12-51.mp4
How can we contact you (optional)
None
Would you like to work on this issue?
Yes
By submitting this issue, I have confirmed that:
The text was updated successfully, but these errors were encountered: