Bug 35532 - Use of calendar for date range in bookings is not clear
Summary: Use of calendar for date range in bookings is not clear
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial (vote)
Assignee: Martin Renvoize
QA Contact: Testopia
URL:
Keywords:
Depends on: 35426
Blocks:
  Show dependency treegraph
 
Reported: 2023-12-10 16:53 UTC by Katrin Fischer
Modified: 2024-03-19 21:55 UTC (History)
8 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
This updates the bookings feature to make selecting the booking period clearer: - Changes field label from 'Period' to 'Booking dates' - Adds a hint added to indicate that you need to select a start and end date ('Select the booking start and end date') - Removes the date shortcut options from the date picker, as they do not make sense for bookings
Version(s) released in:
24.05.00,23.11.04


Attachments
Picker open (35.12 KB, image/png)
2024-01-11 14:50 UTC, Martin Renvoize
Details
Bug 35532: Update date select field and add hint (1.60 KB, patch)
2024-02-19 10:15 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 35532: Remove shortcut buttons from flatpickr (2.38 KB, patch)
2024-02-19 10:32 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 35532: Update date select field and add hint (1.64 KB, patch)
2024-02-19 14:40 UTC, David Nind
Details | Diff | Splinter Review
Bug 35532: Remove shortcut buttons from flatpickr (2.42 KB, patch)
2024-02-19 14:40 UTC, David Nind
Details | Diff | Splinter Review
Bug 35532: Update date select field and add hint (1.70 KB, patch)
2024-03-06 17:47 UTC, Nick Clemens
Details | Diff | Splinter Review
Bug 35532: Remove shortcut buttons from flatpickr (2.48 KB, patch)
2024-03-06 17:47 UTC, Nick Clemens
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Katrin Fischer 2023-12-10 16:53:59 UTC
When making a booking in the staff interface, clicking on the date once won't take, you need to click a second time.
Comment 1 David Nind 2023-12-18 21:11:16 UTC
I can also confirm that this is happening.

To replicate:
1. In the staff interface, go to the details page for a record with items.
2. Click on the 'Items' tab.
3. For an item, change 'Bookable' to 'Yes' and click on 'Update'.
4. Go back to the normal tab for the record.
5. Click on the "Place booking" button.
6. Select a patron for the patron field.
7. Click in the period field.
8. Note that if you select a date (single click as normal for a date field), and then try to press the "Submit" button, the period field is blank.
9. Click on the period field again, then double click a date - this time it remains, and you can click the "Submit" button.
Comment 2 Martin Renvoize 2024-01-11 10:09:02 UTC
You are picking a start AND an end date.. it's not a double click.. it's a start (click one date) then end (click another date).

Katrin suggests a hint should be added to clarify this.
Comment 3 Katrin Fischer 2024-01-11 12:54:17 UTC
Yes, because we didn't get it :D
Comment 4 Martin Renvoize 2024-01-11 14:50:22 UTC
Honestly.. I'm not sure where to put a hint or what to write in it...

To me 'Period' means 'Two dates.. start and end'... and the visual queues are also clear in the picker.
Comment 5 Martin Renvoize 2024-01-11 14:50:48 UTC
Created attachment 160857 [details]
Picker open
Comment 6 David Nind 2024-01-11 22:57:27 UTC
It was not obvious to me.

However, it seems obvious now that you've pointed it out!

I don't think I've seen this pattern elsewhere in Koha (but I'm probably wrong about that...).

Since I'm not familiar with the booking feature, I "assumed" it was just for one day. Normally, with the date picker, I pick a day then click outside the picker so that the date appears in the input field.

So I hope that this didn't come across as too negative!

Some ideas - not tied to any of them:

1. Label for Period: 
   . Start and end date of booking (this is quite long though)
   . Booking dates
   . Start and end date

2. Hint for the field (like for the patron):
   . Select the booking start and end date

3. Not sure how these would work for translation.

I think I prefer a combination of "Booking dates" as the label and a hint for the field "Select the booking start and end date".

Other things noted (outside scope for this bug, just noting - happy to open bugs if they don't exist already):

1. Not sure how to use the 'Tomorrow' date link in the picker - does nothing for me. Should this be disabled?

2. Accessibility things - using keyboard instead of a mouse (I don't know enough to know if these are fixable, or even issues):
   . Patron field: works OK with tab, search, and then the down arrow to select a patron
   . Item field: can tab to it, but can't select item using down or up arrows
   . Period: tab opens date picker, but not sure how to select a date range

3. You need to pick a date range: what if you only wanted to book for one day?
Comment 7 Martin Renvoize 2024-02-19 10:15:00 UTC
Created attachment 162266 [details] [review]
Bug 35532: Update date select field and add hint

This patch updates the 'Period' select from 'Period' to 'Booking dates'
and adds a hint beneath it too.
Comment 8 Martin Renvoize 2024-02-19 10:32:33 UTC
Created attachment 162267 [details] [review]
Bug 35532: Remove shortcut buttons from flatpickr

It doesn't make sense to have the shortcut buttons in the range picker
for bookings. This patch adds the ability to remove them entirely from
display and uses that option in the bookings modal.
Comment 9 David Nind 2024-02-19 14:40:56 UTC
Created attachment 162274 [details] [review]
Bug 35532: Update date select field and add hint

This patch updates the 'Period' select from 'Period' to 'Booking dates'
and adds a hint beneath it too.

Signed-off-by: David Nind <david@davidnind.com>
Comment 10 David Nind 2024-02-19 14:40:58 UTC
Created attachment 162275 [details] [review]
Bug 35532: Remove shortcut buttons from flatpickr

It doesn't make sense to have the shortcut buttons in the range picker
for bookings. This patch adds the ability to remove them entirely from
display and uses that option in the bookings modal.

Signed-off-by: David Nind <david@davidnind.com>
Comment 11 David Nind 2024-02-19 14:41:26 UTC
Thanks Martin!
Comment 12 Nick Clemens 2024-03-06 17:47:47 UTC
Created attachment 162844 [details] [review]
Bug 35532: Update date select field and add hint

This patch updates the 'Period' select from 'Period' to 'Booking dates'
and adds a hint beneath it too.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 13 Nick Clemens 2024-03-06 17:47:49 UTC
Created attachment 162845 [details] [review]
Bug 35532: Remove shortcut buttons from flatpickr

It doesn't make sense to have the shortcut buttons in the range picker
for bookings. This patch adds the ability to remove them entirely from
display and uses that option in the bookings modal.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 14 Katrin Fischer 2024-03-07 12:54:30 UTC
Even I couldn't miss it now, thanks :)
Comment 15 Katrin Fischer 2024-03-07 14:04:05 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 16 Fridolin Somers 2024-03-11 09:13:27 UTC
Depends on Bug 35426 not in 23.11.x
Comment 17 Katrin Fischer 2024-03-11 21:57:10 UTC
(In reply to Fridolin Somers from comment #16)
> Depends on Bug 35426 not in 23.11.x

Maye worth reconsidering - a lot of us didn't understand this new type of date selector and this improves things quite a bit.
Comment 18 Fridolin Somers 2024-03-12 08:14:07 UTC
(In reply to Katrin Fischer from comment #17)
> (In reply to Fridolin Somers from comment #16)
> > Depends on Bug 35426 not in 23.11.x
> 
> Maye worth reconsidering - a lot of us didn't understand this new type of
> date selector and this improves things quite a bit.

If needed for 23.11.x please provide new patches
Comment 19 Fridolin Somers 2024-03-19 08:26:23 UTC
Pushed to 23.11.x for 23.11.04
Comment 20 Fridolin Somers 2024-03-19 08:28:08 UTC
(In reply to Fridolin Somers from comment #18)
> (In reply to Katrin Fischer from comment #17)
> > (In reply to Fridolin Somers from comment #16)
> > > Depends on Bug 35426 not in 23.11.x
> > 
> > Maye worth reconsidering - a lot of us didn't understand this new type of
> > date selector and this improves things quite a bit.
> 
> If needed for 23.11.x please provide new patches

No need, I understood the conflict.
Now calendar.inc is identical between 23.11.x and master
Comment 21 Lucas Gass 2024-03-19 21:55:09 UTC
Missing 23.05.x dependencies, no backport.