Summary: | columns_settings.pl: Circulation issues-table - cannot toggle checkin column | ||
---|---|---|---|
Product: | Koha | Reporter: | Chad Roseburg <croseburg> |
Component: | System Administration | Assignee: | Bugs List <koha-bugs> |
Status: | RESOLVED DUPLICATE | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | cbrannon, george, gmcharlt, gwilliams, jonathan.druart, nick |
Version: | 3.22 | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: |
Description
Chad Roseburg
2015-12-30 00:01:27 UTC
+1 to Chad's suggestion. We have had to hide this column for years with JQuery for the reasons Chad explains. It would be great to hide it through the new columns configuration. Just poking around here, making the columns configurable is easily done, but the 'Renew or check in selected items' needs to be altered too if you can hide checkin. I am wondering if this would be better done as a syspref or as a staff permission? We hide the Check in column with css (display:none) instead of removing the column. There can be issues with removing a column completely. This is the simplest method we found. Anyway, regarding the button, we simply relabel the button with jQuery. Is it possible to modify the button code to change the label or hide it altogether based on which columns are visible? So, it would be code right where the button is displayed that would determine what is on the label. For example, * Button normally displays "Renew or check in selected" (The shorter the better) * Button displays "Renew selected" if the Check in column is hidden * Button displays "Check in selected" if Renew column is hidden * Button is hidden if both Renew and Check in columns are hidden (up for discussion if the Renew all button should also be hidden if the renew column is hidden) I don't think it is wise to have an option for a particular column in a separate preference. It makes sense to have it all together like this. Keeps it simple. |