When deleting items in batch, there is an option to delete the bibliographic record if all the items are deleted. In the new design, the text and the checkbox for this option are misaligned. They are also on the same line as the button to delete the item, which was not the case before. To test: 1. Go to a bibliographic record that has items 2. Click Edit > Delete items in a batch --> At the bottom of the resulting screen, there is a checkbox 'Delete records if no items remain', which is misaligned
Created attachment 143209 [details] Screenshot of the items batch deletion tool in the old design This is a screenshot of the items batch deletion tool in the old design. Note that the text and checkbox under the table are aligned, and also on their own line.
Created attachment 143210 [details] Screenshot of the items batch deletion tool in the new design This is a screenshot of the items batch deletion tool in the new design. Note that the checkbox and text are misaligned and also on the same line as the button and cancel option.
I just noticed that the line break is not in the more recent versions. I was comparing with the manual screenshot, which dates back to I don't know when, but I checked in a 21.05 instance and the option is on the same line as the button and delete option. I kind of like when it is on two lines, but I don't know why it was changed, and maybe there is a reason it was put on the same line...
The checkbox and the label now align correctly on maser - I believe this to be a positive side-effect from some other fixes. We the Save button is still on the same line as the checkbox, that might be something to still look into here. Adjusting bug description.
Created attachment 143920 [details] [review] Bug 32112: Move 'Delete selected items' to its own line This moves the checkbox and its label out of the action field set which makes it appear above the bottons instead of in front of it. To test: * Go to any record * Select items and send to item batch delete * Verify checkbox appears in front of button * Apply patch * Verify the checkbox and the yellow 'delete selected items' button are now on separate lines
Created attachment 143931 [details] [review] Bug 32112: Move 'Delete selected items' to its own line This moves the checkbox and its label out of the action field set which makes it appear above the bottons instead of in front of it. To test: * Go to any record * Select items and send to item batch delete * Verify checkbox appears in front of button * Apply patch * Verify the checkbox and the yellow 'delete selected items' button are now on separate lines Signed-off-by: David Nind <david@davidnind.com>
Created attachment 143944 [details] [review] Bug 32112: Move 'Delete selected items' to its own line This moves the checkbox and its label out of the action field set which makes it appear above the bottons instead of in front of it. To test: * Go to any record * Select items and send to item batch delete * Verify checkbox appears in front of button * Apply patch * Verify the checkbox and the yellow 'delete selected items' button are now on separate lines Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Pushed to master for 22.11. Nice work everyone, thanks!