Skip to content
This repository has been archived by the owner on Dec 7, 2021. It is now read-only.

minute/hour functionality of Select Time doesn't work as expected #98

Open
criemen opened this issue Apr 23, 2013 · 5 comments · May be fixed by #116
Open

minute/hour functionality of Select Time doesn't work as expected #98

criemen opened this issue Apr 23, 2013 · 5 comments · May be fixed by #116

Comments

@criemen
Copy link

criemen commented Apr 23, 2013

Hello,
i've updated my install to the latest HEAD, and now it includes the Select Date functionality (which is great, thanks!). Unfortunately, whenever I use the widget for selecting a Datetime, and move the minute or hour slider, both the from and the until field are set to the same date, rendering the hour and minute selection useless.
The expected behauviour is obviously that only the field which has the focus is updated.

@bwhaley
Copy link
Collaborator

bwhaley commented Apr 25, 2013

Hi,

I'm not able to reproduce this. I click on the Select Date option, then click the first field and pick a date + slider to adjust time, then click Done, then repeat for the second field, then click show, and Voila! It adds /time/04%3A00_20130416/06%3A00_20130416 to the URL and reflects the time correctly in the graph.

@criemen
Copy link
Author

criemen commented Apr 30, 2013

Sorry for not answering earlier, I was on vacation:
The workflow first choosing from, then until works actually fine, I missed that when reporting the bug.
What doesn't work is afterwards adjusting the from time to an earlier (or later, but before until) time, because that sets until always to the same time which is annoying

@hamann
Copy link
Contributor

hamann commented May 23, 2013

Should be fixed in hamann/gdash@7441c05

@Corni does that work for you?

@criemen
Copy link
Author

criemen commented Aug 12, 2013

Sorry for replying so late:
This still does not fix the issue - just play around with selecting a form date, and then playing with until.
Sometimes, setting until or leaving the dialog sets from to the chosen until value, even with the patch.
Aditionally, the same behaviour can be triggered by adjusting from - it overwrites until as well.

@criemen
Copy link
Author

criemen commented Aug 12, 2013

Sorry for the previous comment, I did not save the edited file, so I didn't apply the patch.
It works great, please merge this to trunk!
Thanks for your efforts 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants