While integrating bug 10860 into current master we removed the change to the prog template it included as the template had just been removed. The changed intended for prog should be reimplemented for the bootstrap theme.
I have almost a patch for this one. But at the OPAC, tabs already exist ("show all items" and "showing last 50 items"). I am sure it's not a good idea to add a 2 level tabs here. An ergonomic suggestion? I would replace the existing tab by a link.
Hi Jonathan, maybe we could split this up - show the in-house checkouts in the patron account and show it on the OPAC detail tab? Bit confused what we are talking about right now :)
Created attachment 33488 [details] [review] Bug 13234: On-site checkouts - OPAC This patch introduces the code lost in bug 10860 for the OPAC side. Test plan: Go on opac-readingrecord.pl and verify the tabs work as expected and the "show all items" and "show 50 items" links.
Created attachment 33497 [details] [review] [SIGNED-OFF] Bug 13234: On-site checkouts - OPAC This patch introduces the code lost in bug 10860 for the OPAC side. Test plan: Go on opac-readingrecord.pl and verify the tabs work as expected and the "show all items" and "show 50 items" links. Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 33498 [details] [review] Bug 13234 [Follow-up] Make on-site checkouts visible in OPAC This follow-up makes a few changes to the template: 1. The "show all" / "show 50" links have been modified to show the current state. 2. The tabs are only shown if the OnSiteCheckouts preference is turned on. 3. The DataTables configuration has been modified so that title sorting ignores articles, sorting on the the first column is disabled, and sorting by date works regardless of your dateformat preference. 4. Some indentation has been corrected and markup comments added. To test the opacreadinghistory preference must be enabled. Log in to the OPAC as a patron who has some on-site checkouts as well as regular checkouts. With OnSiteCheckouts enabled, view the reading history page and confirm that the tabs work correctly. Test the table sorting controls. With OnSiteCheckouts disabled, confirm that the tabs do not appear. Test the "Show all items"/"Show last 50 items" links and confirm that the behavior is correct.
Created attachment 33515 [details] [review] Bug 13234 [Follow-up] Make on-site checkouts visible in OPAC This follow-up makes a few changes to the template: 1. The "show all" / "show 50" links have been modified to show the current state. 2. The tabs are only shown if the OnSiteCheckouts preference is turned on. 3. The DataTables configuration has been modified so that title sorting ignores articles, sorting on the the first column is disabled, and sorting by date works regardless of your dateformat preference. 4. Some indentation has been corrected and markup comments added. To test the opacreadinghistory preference must be enabled. Log in to the OPAC as a patron who has some on-site checkouts as well as regular checkouts. With OnSiteCheckouts enabled, view the reading history page and confirm that the tabs work correctly. Test the table sorting controls. With OnSiteCheckouts disabled, confirm that the tabs do not appear. Test the "Show all items"/"Show last 50 items" links and confirm that the behavior is correct. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Owen, Thanks for this follow-up! Now we have the sort available in the table, should the "order by" dropdown list be removed?
Created attachment 33516 [details] [review] Bug 13234: Remove the order by control Now we have the sort available in the table, the "order by" dropdown list can be removed.
Owen, Could you please sign off this last patch or obsolete it if you disagree?
(In reply to Jonathan Druart from comment #8) > Now we have the sort available in the table, the "order by" dropdown list can > be removed. Removing the dropdown would remove that functionality for users without JavaScript, so I would rather we not remove it. However, we could hide it with JavaScript or add this to the OPAC stylesheet: .no-js #opac-user-readingrec .resultscontrol { display: none; }
ok, patch obsoleted.
Created attachment 33598 [details] [review] [PASSED QA] Bug 13234: On-site checkouts - OPAC This patch introduces the code lost in bug 10860 for the OPAC side. Test plan: Go on opac-readingrecord.pl and verify the tabs work as expected and the "show all items" and "show 50 items" links. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 33599 [details] [review] [PASSED QA] Bug 13234 [Follow-up] Make on-site checkouts visible in OPAC This follow-up makes a few changes to the template: 1. The "show all" / "show 50" links have been modified to show the current state. 2. The tabs are only shown if the OnSiteCheckouts preference is turned on. 3. The DataTables configuration has been modified so that title sorting ignores articles, sorting on the the first column is disabled, and sorting by date works regardless of your dateformat preference. 4. Some indentation has been corrected and markup comments added. To test the opacreadinghistory preference must be enabled. Log in to the OPAC as a patron who has some on-site checkouts as well as regular checkouts. With OnSiteCheckouts enabled, view the reading history page and confirm that the tabs work correctly. Test the table sorting controls. With OnSiteCheckouts disabled, confirm that the tabs do not appear. Test the "Show all items"/"Show last 50 items" links and confirm that the behavior is correct. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Works as described, passes tests and QA script. Good addition to the new on-site feature. Note: It would be nice to show the 'on-site' note also in the liste of checkouts on the summary page!
Patches pushed to master. Thanks to you guys!
Note: this was only for the reading history, it appears not to show up for current checkouts - see bug 13557.