Change behavior when changing dates in a datepicker group to be more intuitive. #2583
Conversation
drgrice1
force-pushed
the
datepicker-group-update-tweak
branch
from
8391a24
to
13afbcf
Compare
|
Overall this is what I was asking for in #2581, though I may have not been clear enough there. I just wanted user input to always be honored and the other dates updated based on that. In testing, this doesn't seem to be fully working for me, as the open date is never being adjusted under any of my tests when it should. My tests are to set all the dates (open, reduced, close, answer) to be the same. Then when I set the answer, closed, or reduced date to be an earlier date, the open date isn't updated (all the other dates are updated appropriately though). |
|
The open date updates when I test it. 😜 |
|
You did see that I added another commit after you made the comment? |
|
Yea, I saw the fix, I tried to figure out why the open date was being ignored, but missed the loop index needed changed. |
drgrice1
force-pushed
the
datepicker-group-update-tweak
branch
from
d6c0d02
to
267a826
Compare
|
I seeing the following behavior with the date picker and setting the dates for individual user(s). Go override the dates for a single user, set all the dates to the same date in the future (I'm using a full year). Then delete the 'close' date. When this happens the reduced scoring and open date are reset, but they aren't rest to a valid date corresponding to the set close date. Here is a screen shot.
To me the dates should have reverted to |
|
Note, setting all the dates to some date over a year in the past, and deleting the open date, does reset the reduced, close, and answer date to equal the open date of the set, so I wasn't able to reproduce any issue with updating the dates in the other direction. |
|
Are you testing with this pull request? I just tested with this pull request and used the exact dates shown in your screen shot, and I don't see that. When I delete the close date, the open and reduced scoring dates are then set to |
|
Yes, I'm testing with this PR. Wonder what other settings in my set could be causing the problems. |
|
Oh, I think I figured it out. I have previously overridden the close/answer date for this set/user to be |
…intuitive. Previously when any of the dates were changed, the open, reduced scoring, close, and answer dates were ensured to be in the correct order by always changing the input later in the last to a later date if needed. This instead acts on the input that was modified. That input value is not changed from what the user just changed it to. Any earlier inputs in the list are set to the new date in the changed input if they are later than that date, and any later inputs in the list are set to the new date in the changed input if they are earlier than that date. This is to address issue openwebwork#2581.
Previously the open date was never acted on, and so the for loop started at one which skipped the open date. Now it needs to be acted on.
When using the "Today" or "Now" buttons, the "changed" class is not being added to the input. When the handlers for those buttons are called, the "change" event needs to be triggered so that this happens.
drgrice1
force-pushed
the
datepicker-group-update-tweak
branch
from
c003744
to
28557d8
Compare
Previously when any of the dates were changed, the open, reduced scoring, close, and answer dates were ensured to be in the correct order by always changing the input later in the last to a later date if needed.
This instead acts on the input that was modified. That input value is not changed from what the user just changed it to. Any earlier inputs in the list are set to the new date in the changed input if they are later than that date, and any later inputs in the list are set to the new date in the changed input if they are earlier than that date.
This is to address issue #2581.