Sometime between 20.05 and 21.05 (we skipped 20.11), the name of a patron club is now showing up in the clubs table as hyperlinked. Clicking this hyperlink doesn't do anything. I've tried to search through release notes and bugs to find what the hyperlinking is supposed to do and couldn't trace it. The screenshots in the manual and other online tutorials show that the club name was not previously hyperlinked. The hyperlink could be directed to open the enrollments list or to edit the club's settings. Both of these options are available in the Actions menu. I think it would also be okay to remove the hyperlink altogether to reduce confusion. (I'm almost ashamed to admit how many times I clicked on it thinking it was a shortcut to the enrollments list!) Also, there's a CSS style being applied to the table where every other row is highlighted yellow when the mouse pointer hovers over it. This is distracting and not helpful.
Created attachment 127818 [details] [review] Bug 29521: Fix style of patron clubs table when in clubs management This patch follows up Bug 24627 to add some additional template logic to the table of clubs so that the interface is consistent whether you're looking at clubs during the holds process or during the clubs management view. The ways Bug 24627 made the clubs list more consistent with patron search results made it LESS consistent with the Clubs management interface. To test, apply the patch and go to Tools -> Patron clubs. - Scroll down to the "Clubs" table, below the "Club templates" table. - In the table of clubs, the club name should not be a link. - Hovering your mouse over the table should not highlight each row. Confirm the previous behavior of a clubs search when placing a hold: - Search the catalog and start the process of placing a hold on an item. - On the "Place hold" page, click the "Clubs" tab and perform a search which will return multiple clubs. - In this view, the club name should be a link. Clicking it should select that club for the hold. - You should also be able to click anywhere in the row to select the club for the hold.
Created attachment 127849 [details] [review] Bug 29521: Fix style of patron clubs table when in clubs management This patch follows up Bug 24627 to add some additional template logic to the table of clubs so that the interface is consistent whether you're looking at clubs during the holds process or during the clubs management view. The ways Bug 24627 made the clubs list more consistent with patron search results made it LESS consistent with the Clubs management interface. To test, apply the patch and go to Tools -> Patron clubs. - Scroll down to the "Clubs" table, below the "Club templates" table. - In the table of clubs, the club name should not be a link. - Hovering your mouse over the table should not highlight each row. Confirm the previous behavior of a clubs search when placing a hold: - Search the catalog and start the process of placing a hold on an item. - On the "Place hold" page, click the "Clubs" tab and perform a search which will return multiple clubs. - In this view, the club name should be a link. Clicking it should select that club for the hold. - You should also be able to click anywhere in the row to select the club for the hold. Signed-off-by: Andreas Roussos <a.roussos@dataly.gr>
Created attachment 127864 [details] [review] Bug 29521: Fix style of patron clubs table when in clubs management This patch follows up Bug 24627 to add some additional template logic to the table of clubs so that the interface is consistent whether you're looking at clubs during the holds process or during the clubs management view. The ways Bug 24627 made the clubs list more consistent with patron search results made it LESS consistent with the Clubs management interface. To test, apply the patch and go to Tools -> Patron clubs. - Scroll down to the "Clubs" table, below the "Club templates" table. - In the table of clubs, the club name should not be a link. - Hovering your mouse over the table should not highlight each row. Confirm the previous behavior of a clubs search when placing a hold: - Search the catalog and start the process of placing a hold on an item. - On the "Place hold" page, click the "Clubs" tab and perform a search which will return multiple clubs. - In this view, the club name should be a link. Clicking it should select that club for the hold. - You should also be able to click anywhere in the row to select the club for the hold. Signed-off-by: Andreas Roussos <a.roussos@dataly.gr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 22.05, thanks to everybody involved!
Pushed to 21.11.x for 21.11.01
Added Manual keyword for 22.05 bugs that I consider require documentation changes. See bug 29640 and the linked spreadsheet for additional details.
Pushed to 21.05.x for 21.05.08
Not backported to oldoldstable (20.11.x). Feel free to ask if it's needed.