Bug 35241: Fix markup errors in point of sale template
authorOwen Leonard <oleonard@myacpl.org>
Thu, 2 Nov 2023 18:36:06 +0000 (18:36 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Mon, 6 Nov 2023 14:34:43 +0000 (11:34 -0300)
commitb5477b2ea3fb8f8bf367012ff621e6dd7936eefc
tree73ce84335aa5a754b541d905552208fc1273413c
parent79bf4485c1b2ae8059e2ee15ea1e67bba9961c34
Bug 35241: Fix markup errors in point of sale template

This patch makes several markup corrections to the Point of Sale page,
fixing duplicate ids, invalid "selected" attributes*, a broken comment
tag, and an invalid "href" attribute.

* The "selected" attribute in this template is on a disabled <option>,
  so it can't be selected.

The patch also changes <div class="action"> to <fieldset class="action">
for consistency and better layout.

It makes some changes to the "Payment received" dialog in order to
improve visual consistency.

To test, apply the patch and make sure the EnablePointOfSale preference
is turned on.

- Go to Point of Sale.
- Test the process of adding items for purchase and collecting payment.
- After you submit the payment the confirmation dialog should look
  correct, and the "Print" and "Email" buttons should both work
  correctly.

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/en/modules/pos/pay.tt