Bug 35351: Adjust basket details template to avoid showing empty page-section

This patch makes a minor change to the markup of the basket details page
so that we don't display an empty div.page-section when there are no
orders.

Note: This patch includes whitespace changes, so please ignore
whitespace when viewing the diff.

To test, apply the patch and go to Acquisitions.

- Search for a vendor and add a basket if necessary.
- View the details of an empty basket.
- Under the "General information" or "Settings" sections there should be
  no empty white box.
- View the details of a basket which has orders (using the "Add to
  basket" process if necessary).
  - Orders should be displayed correctly.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
This commit is contained in:
Owen Leonard 2023-11-16 14:12:17 +00:00 committed by Katrin Fischer
parent d485ff32b0
commit dde374e2ad
Signed by: kfischer
GPG key ID: 0EF6E2C03357A834

View file

@ -482,8 +482,8 @@
</div>
[% END %]
<div id="acqui_basket_content" class="page-section">
[% IF ( books_loop ) %]
[% IF ( books_loop ) %]
<div id="acqui_basket_content" class="page-section">
<h2>Orders</h2>
<table id="orders">
<thead>
@ -755,9 +755,9 @@
[% END %]
</tbody>
</table> <!-- /#orders -->
[% END # / IF ( books_loop ) %]
[% IF ( listincgst ) %]<small class="highlight">** Vendor's listings already include tax.</small>[% END %]
</div> <!-- /#acqui_basket_content -->
[% IF ( listincgst ) %]<small class="highlight">** Vendor's listings already include tax.</small>[% END %]
</div> <!-- /#acqui_basket_content -->
[% END # / IF ( books_loop ) %]
[% IF (cancelledorders_loop) %]
<div id="cancelledorders" class="page-section">