Summary: | Circulation tabs inconsistent with counters | ||
---|---|---|---|
Product: | Koha | Reporter: | Christopher Brannon <cbrannon> |
Component: | Circulation | Assignee: | Owen Leonard <oleonard> |
Status: | ASSIGNED --- | QA Contact: | Martin Renvoize (ashimema) <martin.renvoize> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | david, george, gmcharlt, kyle.m.hall |
Version: | Main | Keywords: | rel_24_05_candidate |
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25430 | ||
Change sponsored?: | Sponsored | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: |
This enhancement changes the markup, style, and behaviour of counters which are displayed on a patron's check out and details tabs:
1. A new style: the counter background has a light blue square with rounded corners
2. The claims tab:
. The order of the counters is now Unresolved claims / Resolved claims
. The background colour for the counter changes to orange when the value set in ClaimReturnedWarningThreshold is reached
3. The restrictions tab: the counter background color is red
4. The clubs tab: now only shows the number of clubs a patron is enrolled in (the count of available clubs is removed)
|
Version(s) released in: | |
Circulation function: | |||
Bug Depends on: | |||
Bug Blocks: | 24097 | ||
Attachments: |
Inconsistencies in tabs
Updated inconsistencies An idea for updated tab styles Bug 26258: Circulation tabs inconsistent with counters Bug 26258: Circulation tabs inconsistent with counters Screenshots - Patron check out and details tabs (before and after) Bug 26258: Circulation tabs inconsistent with counters |
Description
Christopher Brannon
2020-08-19 20:57:48 UTC
And club counts should not be in parenthesis. Created attachment 108694 [details]
Inconsistencies in tabs
I agree. Consistency would be nice. *** Bug 26264 has been marked as a duplicate of this bug. *** Created attachment 108779 [details]
Updated inconsistencies
After some discussion on IRC we all believed it would be best to move towards using the styling from the new 'Claims' tab consistently on all the tabs here. i.e. Tab name followed by label span. (With the label span being colour reactive if there is a state to convey) I think it's worth reconsidering whether claims and clubs really need to show the "inactive" count. Why should we need to know the number of clubs available? Why should we know how many inactive claims there are? (In reply to Owen Leonard from comment #7) > I think it's worth reconsidering whether claims and clubs really need to > show the "inactive" count. Why should we need to know the number of clubs > available? Why should we know how many inactive claims there are? +1 We could have grammatically-correct labels using the techniques added by Bug 15395, and could unify the different labels by showing a count of only unresolved claims and only enrolled clubs. https://zivotdesign.com/p/#1/15980387474009 I agree on the clubs. I could go either way on the claims. I would think that staff would care about how many claims altogether AND how many are unresolved. Other than that, I think it is a great idea. I think the first letter should be all caps, even though the number is first. It just looks unfinished otherwise. It bugs me. I like it, especially the idea for making it work for singular/plural and getting rid of the (s). How's this looking now with the staff client refresh? At least on our test server, the claims counts are displayed differently than the other counters. They are incorporating labels that the others do not. They are at least consistent now. I agree with Owen that I don't think it is necessary to know how many clubs are available, just how many they are signed up for. I could still go either way on having both numbers on the inactive claims. Maybe just have the number of claims, and if something needs attention, change the color? I think if we are going to move towards using labels, all the counters should be in labels, and we should do away with the parenthesis. It seems overkill. One or the other. Labels look better and provide more functionality. Created attachment 153679 [details]
An idea for updated tab styles
(In reply to Owen Leonard from comment #14) > Created attachment 153679 [details] > An idea for updated tab styles I think that looks great! Should the dual counters be in the same label? Maybe you want problematic counts to be red or something that stands in contrast to the good numbers? Does that make sense? (In reply to Owen Leonard from comment #14) > Created attachment 153679 [details] > An idea for updated tab styles +1 I'd love to see this submitted [U+1F642] It went to assigned... That's got me excited [U+1F60A] Exciting times. :) Open, I like your concept. I still think we should do away with the total number of clubs. It is arbitrary. Not all clubs apply to all patrons. It isn't really useful in this scenario. Created attachment 166147 [details] [review] Bug 26258: Circulation tabs inconsistent with counters This patch makes changes to the markup, style, and behaviour of counters which are displayed on the patron's account tabs: Checkouts, Holds, etc. Tabs which contain a count which might change based on a user's actions will keep up to date, e.g. changing claims status, enrolling in clubs, etc. To test, apply the patch and enable features if necessary: recalls, article requests, claims, clubs, and patron lists. Test both from the checkout page and the patron detail page: Recalls: - If necessary, check out some items to patrons so that there are titles which can be recalled. - Log in to the OPAC and place one or more recalls. - Return to the staff interface and view the "Recalls" tab on the patron's account. - Click Actions -> Cancel for one of the recalls. The recall should be marked "Cancelled" and the count on the tab should update. Return claims: - If necessary, check out some items to a patron. Click the "Claim returned" button on a few of the patron's checkouts. - As you add claims, the "Claims" tab should update the count of claims. *** NOTE: This changes the display of claims in the tab so that the count of unresolved claims appears first: "Unresolved / Resolved". I think it makes more sense for the first number to be the one which shows the number of "active" issues. - Open the Claims tab. Click Actions->Resolve on one or more claims, confirming that the resolved claim is hidden and the tab numbers update. - Test setting the ClaimReturnedWarningThreshold system preference to various values: e.g. empty, 1, 5. - If the pref is empty, OR if the pref is set to a number higher than the number of patron claims, the count of claims should be styled with the "info" blue background and the count of resolved claims is not shown. *** NOTE: Hiding the resolved claims seems logical if there is not a concern about the total number of claims. - If the number of all claims exceeds the value of the preference, the tab should show a count of "Unresolved claims / Resolved claims" with the yellow warning style. Restrictions: - Adding a restriction to the account should cause a counter with a red background to appear on the tab. Clubs: - If necessary, create at least one club template and then multiple clubs. - From the "Clubs" tab on the patron's account, try enrolling and cancelling enrollments to confirm that the count on the tab updates correctly. *** NOTE: I have removed the count of available clubs because I think it's not relevant to this interface. We don't do it with patron lists, so this makes it a little more consistent.. Patron lists: - If necessary, create two ore more patron lists. - From the "Patron lists" tab, try adding and removing the patron from lists. The tab counter should update correctly. Sponsored-by: Athens County Public Libraries Created attachment 166159 [details] [review] Bug 26258: Circulation tabs inconsistent with counters This patch makes changes to the markup, style, and behaviour of counters which are displayed on the patron's account tabs: Checkouts, Holds, etc. Tabs which contain a count which might change based on a user's actions will keep up to date, e.g. changing claims status, enrolling in clubs, etc. To test, apply the patch and enable features if necessary: recalls, article requests, claims, clubs, and patron lists. Test both from the checkout page and the patron detail page: Recalls: - If necessary, check out some items to patrons so that there are titles which can be recalled. - Log in to the OPAC and place one or more recalls. - Return to the staff interface and view the "Recalls" tab on the patron's account. - Click Actions -> Cancel for one of the recalls. The recall should be marked "Cancelled" and the count on the tab should update. Return claims: - If necessary, check out some items to a patron. Click the "Claim returned" button on a few of the patron's checkouts. - As you add claims, the "Claims" tab should update the count of claims. *** NOTE: This changes the display of claims in the tab so that the count of unresolved claims appears first: "Unresolved / Resolved". I think it makes more sense for the first number to be the one which shows the number of "active" issues. - Open the Claims tab. Click Actions->Resolve on one or more claims, confirming that the resolved claim is hidden and the tab numbers update. - Test setting the ClaimReturnedWarningThreshold system preference to various values: e.g. empty, 1, 5. - If the pref is empty, OR if the pref is set to a number higher than the number of patron claims, the count of claims should be styled with the "info" blue background and the count of resolved claims is not shown. *** NOTE: Hiding the resolved claims seems logical if there is not a concern about the total number of claims. - If the number of all claims exceeds the value of the preference, the tab should show a count of "Unresolved claims / Resolved claims" with the yellow warning style. Restrictions: - Adding a restriction to the account should cause a counter with a red background to appear on the tab. Clubs: - If necessary, create at least one club template and then multiple clubs. - From the "Clubs" tab on the patron's account, try enrolling and cancelling enrollments to confirm that the count on the tab updates correctly. *** NOTE: I have removed the count of available clubs because I think it's not relevant to this interface. We don't do it with patron lists, so this makes it a little more consistent.. Patron lists: - If necessary, create two ore more patron lists. - From the "Patron lists" tab, try adding and removing the patron from lists. The tab counter should update correctly. Sponsored-by: Athens County Public Libraries Signed-off-by: David Nind <david@davidnind.com> I'll attach some before and after screenshots. My testing notes (using KTD): 1. Recalls - Enable system preference - Set some rules: . Recalls allowed (total): 5 . Recalls per record (count): 5 . On shelf recalls allowed: Recall due date interval (day) . Recall overdue fine amount: 5 . Recall pickup period (day): 5 - Checkout some items to Patron A - Log in as Patron B (set username/password by editing the patron details) - Make a recall for Patron B for one of the items checked out by Patron A 2. Returns claimed: - Select a value for ClaimReturnedLostValue for the claim lost action button to appear under a patron's checkouts: I used Lost - Set a value for ClaimReturnedWarningThreshold: 2 - Check out some items to the patron - Make 3 claims - Resolve one claim: Claims tab > Actions > Resolve Created attachment 166160 [details]
Screenshots - Patron check out and details tabs (before and after)
Screenshots of a patron's check out and details tabs, before and after the patch.
Created attachment 166239 [details] [review] Bug 26258: Circulation tabs inconsistent with counters This patch makes changes to the markup, style, and behaviour of counters which are displayed on the patron's account tabs: Checkouts, Holds, etc. Tabs which contain a count which might change based on a user's actions will keep up to date, e.g. changing claims status, enrolling in clubs, etc. To test, apply the patch and enable features if necessary: recalls, article requests, claims, clubs, and patron lists. Test both from the checkout page and the patron detail page: Recalls: - If necessary, check out some items to patrons so that there are titles which can be recalled. - Log in to the OPAC and place one or more recalls. - Return to the staff interface and view the "Recalls" tab on the patron's account. - Click Actions -> Cancel for one of the recalls. The recall should be marked "Cancelled" and the count on the tab should update. Return claims: - If necessary, check out some items to a patron. Click the "Claim returned" button on a few of the patron's checkouts. - As you add claims, the "Claims" tab should update the count of claims. *** NOTE: This changes the display of claims in the tab so that the count of unresolved claims appears first: "Unresolved / Resolved". I think it makes more sense for the first number to be the one which shows the number of "active" issues. - Open the Claims tab. Click Actions->Resolve on one or more claims, confirming that the resolved claim is hidden and the tab numbers update. - Test setting the ClaimReturnedWarningThreshold system preference to various values: e.g. empty, 1, 5. - If the pref is empty, OR if the pref is set to a number higher than the number of patron claims, the count of claims should be styled with the "info" blue background and the count of resolved claims is not shown. *** NOTE: Hiding the resolved claims seems logical if there is not a concern about the total number of claims. - If the number of all claims exceeds the value of the preference, the tab should show a count of "Unresolved claims / Resolved claims" with the yellow warning style. Restrictions: - Adding a restriction to the account should cause a counter with a red background to appear on the tab. Clubs: - If necessary, create at least one club template and then multiple clubs. - From the "Clubs" tab on the patron's account, try enrolling and cancelling enrollments to confirm that the count on the tab updates correctly. *** NOTE: I have removed the count of available clubs because I think it's not relevant to this interface. We don't do it with patron lists, so this makes it a little more consistent.. Patron lists: - If necessary, create two ore more patron lists. - From the "Patron lists" tab, try adding and removing the patron from lists. The tab counter should update correctly. Sponsored-by: Athens County Public Libraries Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> I am not completely sure about the color choice in combination with the dark green of the tabs. But I also see something "blue" on the restrictions and patron list tabs that I am not sure about (screenshot in Mattermost). Can you please have a look? I am sorry, I still get the little blue boxes :( More so, I pushed another patron accounts tab: bug 33737 - Add bookings to patron details - could you please include a follow-up for it? Okay, going off the information that Katrin is using Firefox 115 ESR I realized that my patch uses some CSS which isn't widely-supported enough to work for us yet. The :has pseudo class only arrived in Firefox as of v121. Using a combination of :has and :empty let me simplify the template logic a lot but I can try to do it the hard way. I really like where you were going with this :) |