Bug 34209 - Follow up on Bug 28726 - move whole search header div into checkbox column condition
Summary: Follow up on Bug 28726 - move whole search header div into checkbox column co...
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Label/patron card printing (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial (vote)
Assignee: Michael Hafen
QA Contact: Katrin Fischer
URL:
Keywords:
Depends on: 28726
Blocks:
  Show dependency treegraph
 
Reported: 2023-07-05 17:25 UTC by Michael Hafen
Modified: 2023-09-13 20:38 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.11.00,23.05.02,22.11.08


Attachments
Bug 28726: Follow up - move whole 'select all ...' div into checkbox column condition (1.75 KB, patch)
2023-07-05 17:25 UTC, Michael Hafen
Details | Diff | Splinter Review
Bug 34209: Follow up on Bug 28726 - move whole 'select all ...' header into condition (1.77 KB, patch)
2023-07-12 11:29 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 34209: Follow up on Bug 28726 - move whole 'select all ...' header into condition (2.20 KB, patch)
2023-07-12 21:16 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Hafen 2023-07-05 17:25:20 UTC
Created attachment 153075 [details] [review]
Bug 28726: Follow up - move whole 'select all ...' div into checkbox column condition

In the patron search popup from pages that don't have a checkbox in the search results table there is an empty div above the table.  This div holds the 'Select all ...' features when there is a checkbox column in the table.
Comment 1 Jonathan Druart 2023-07-11 06:55:24 UTC
Thanks, Michael.
Comment 2 Owen Leonard 2023-07-12 11:29:11 UTC
Created attachment 153350 [details] [review]
Bug 34209: Follow up on Bug 28726 - move whole 'select all ...' header into condition

Move the whole search header div into the checkbox column condition, so
that other patron search popups don't have an empty div.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 3 Katrin Fischer 2023-07-12 21:16:20 UTC
Created attachment 153388 [details] [review]
Bug 34209: Follow up on Bug 28726 - move whole 'select all ...' header into condition

Move the whole search header div into the checkbox column condition, so
that other patron search popups don't have an empty div.

To test:
* Activate the ERM module
* Create a new agreement
* Search for a user using the pop-up search
* Verify there is an empty div showing as grey box above the table
* Go to tools > label creator > new batch
* Search for patrons
* Verify the box isn't empty and shows the select all ... options
* Apply patch
* Repeat steps, verify the empty div is gone and the options remain

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Katrin Fischer 2023-07-12 21:16:43 UTC
Please make sure to add a test plan to your patches, working out what I need to test/look at without many hints costs time I cannot spend elsewhere.
Comment 5 Tomás Cohen Arazi 2023-07-14 18:24:20 UTC
Pushed to master for 23.11.

Nice work everyone, thanks!
Comment 6 Martin Renvoize 2023-07-18 11:52:26 UTC
Thanks for all the hard work!

Pushed to 23.05.x for the next release
Comment 7 Matt Blenkinsop 2023-07-18 15:29:05 UTC
Nice work everyone!

Pushed to oldstable for 22.11.x