Bug 36295 - Space out content blocks in batch record deletion
Summary: Space out content blocks in batch record deletion
Status: Pushed to oldstable
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial
Assignee: Owen Leonard
QA Contact: Martin Renvoize (ashimema)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-03-11 20:36 UTC by Caroline Cyr La Rose
Modified: 2024-07-25 11:11 UTC (History)
4 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:
24.05.00,23.11.06,23.05.12
Circulation function:


Attachments
Bug 36295: Space out content blocks in batch record deletion (1.25 KB, patch)
2024-04-09 12:07 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 36295: Space out content blocks in batch record modification (1.27 KB, patch)
2024-04-12 16:44 UTC, Brendan Lawlor
Details | Diff | Splinter Review
Bug 36295: Space out content blocks in batch record deletion (1.32 KB, patch)
2024-04-15 15:00 UTC, Caroline Cyr La Rose
Details | Diff | Splinter Review
Bug 36295: Space out content blocks in batch record modification (1.34 KB, patch)
2024-04-15 15:00 UTC, Caroline Cyr La Rose
Details | Diff | Splinter Review
Bug 36295: Space out content blocks in batch record deletion (1.39 KB, patch)
2024-04-23 15:31 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review
Bug 36295: Space out content blocks in batch record modification (1.41 KB, patch)
2024-04-23 15:31 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Caroline Cyr La Rose 2024-03-11 20:36:15 UTC
This is very minor, but I figure I'd report it anyway... When using the batch record modification tool, there is no space between the "Record type" content block and the tabbed "Source of records" block.

To recreate:
1. Create a MARC modification template
   1.1. Go to Cataloging > MARC modification templates > New template
   1.2. Give the template a name (for example, "Hide from OPAC")
   1.3. Click Submit
   1.4. Click New action
   1.5. Add a new action (for example, Update existing or add new field(s) 942 n with value 1)
   1.6. Click Add action

2. Go to Cataloging > Batch record modification
   --> Note that there is no space between the white "Record type" block and the block underneath with the tabs for the record source
Comment 1 Caroline Cyr La Rose 2024-03-11 20:36:57 UTC
I found this in 23.05.
Comment 2 Owen Leonard 2024-04-09 11:59:11 UTC
I see this on the Batch record deletion page but not the batch record modification page.
Comment 3 Owen Leonard 2024-04-09 12:07:34 UTC
Created attachment 164554 [details] [review]
Bug 36295: Space out content blocks in batch record deletion

This patch adds a float-clearing line break to the batch record deletion
template so that the "Record type" section has visual separation from
the record number submission tabs.

To test, apply the patch and go to Cataloging -> Batch record deletion.

There should be padding between the two sections.
Comment 4 Brendan Lawlor 2024-04-12 16:27:33 UTC
In Chrome I see the missing padding in batch record modification too but not in Firefox.

In batch_record_modification.tt the spacing was written as <br><br> but changing it to <br class="clear"/> fixes it for Chrome.
Comment 5 Brendan Lawlor 2024-04-12 16:44:34 UTC
Created attachment 164860 [details] [review]
Bug 36295: Space out content blocks in batch record modification

This patch adds a float clearing class to a line break in the batch record modification template to fix a spacing issue in Chrome.

To test apply the patch and go to Cataloging -> Batch record modification

There should be space between the two sections.
Comment 6 Caroline Cyr La Rose 2024-04-15 15:00:39 UTC
Created attachment 164889 [details] [review]
Bug 36295: Space out content blocks in batch record deletion

This patch adds a float-clearing line break to the batch record deletion
template so that the "Record type" section has visual separation from
the record number submission tabs.

To test, apply the patch and go to Cataloging -> Batch record deletion.

There should be padding between the two sections.

Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
Comment 7 Caroline Cyr La Rose 2024-04-15 15:00:41 UTC
Created attachment 164890 [details] [review]
Bug 36295: Space out content blocks in batch record modification

This patch adds a float clearing class to a line break in the batch record modification template to fix a spacing issue in Chrome.

To test apply the patch and go to Cataloging -> Batch record modification

There should be space between the two sections.

Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
Comment 8 Caroline Cyr La Rose 2024-04-15 15:01:05 UTC
Thanks all! Looks much better
Comment 9 Martin Renvoize (ashimema) 2024-04-23 15:31:54 UTC
Created attachment 165407 [details] [review]
Bug 36295: Space out content blocks in batch record deletion

This patch adds a float-clearing line break to the batch record deletion
template so that the "Record type" section has visual separation from
the record number submission tabs.

To test, apply the patch and go to Cataloging -> Batch record deletion.

There should be padding between the two sections.

Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 10 Martin Renvoize (ashimema) 2024-04-23 15:31:57 UTC
Created attachment 165408 [details] [review]
Bug 36295: Space out content blocks in batch record modification

This patch adds a float clearing class to a line break in the batch record modification template to fix a spacing issue in Chrome.

To test apply the patch and go to Cataloging -> Batch record modification

There should be space between the two sections.

Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 11 Katrin Fischer 2024-04-26 12:17:04 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 12 Fridolin Somers 2024-05-24 07:35:28 UTC
Pushed to 23.11.x for 23.11.06
Comment 13 Lucas Gass (lukeg) 2024-05-28 20:39:23 UTC
Backported to 23.05.x for upcoming 23.05.12