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.

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

TimePicker minutes are always set to current time #890

Closed
ialokim opened this issue Nov 14, 2023 · 1 comment · Fixed by #893
Closed

TimePicker minutes are always set to current time #890

ialokim opened this issue Nov 14, 2023 · 1 comment · Fixed by #893
Assignees
@Altonss
Labels
beginner job 🔰 This is a relatively easy task suitable for new contributors. bug 🐞 A functional defect or unexpected behavior.

Comments

@ialokim
Copy link
Collaborator

ialokim commented Nov 14, 2023

Describe the bug
The minute part of the time picker in the trip search always defaults to the current time instead of the previously selected one.

To Reproduce
Steps to reproduce the behavior:

  1. search for any connection
  2. change the departure/arrival time, especially the minute part
  3. try to change the time again, notice that the minute part is automatically adjusted to the current time

Expected behavior
Both the hour and minute part of the time picker should coincide with the previously selected time.

Versions (please complete the following information):

  • Transportr Version: 2.2.0 (preview)
  • Device: Google Pixel 4a
  • Android Version: 13
@ialokim ialokim added the bug 🐞 A functional defect or unexpected behavior. label Nov 14, 2023
@ialokim ialokim mentioned this issue Nov 14, 2023
Merged
11 tasks
@Altonss
Copy link
Collaborator

Altonss commented Nov 14, 2023

This issue is already present in 2.1.5

@Altonss Altonss mentioned this issue Dec 6, 2023
Merged
@Altonss Altonss self-assigned this Dec 6, 2023
@Altonss Altonss linked a pull request Dec 6, 2023 that will close this issue
Fix timePicker minutes always set to current time #893
Merged
@Altonss Altonss added the beginner job 🔰 This is a relatively easy task suitable for new contributors. label Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Altonss Altonss

Labels
beginner job 🔰 This is a relatively easy task suitable for new contributors. bug 🐞 A functional defect or unexpected behavior.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix timePicker minutes always set to current time grote/Transportr
2 participants
@ialokim @Altonss