Bug 34934: Remove the use of event attributes from OPAC lists page
authorOwen Leonard <oleonard@myacpl.org>
Wed, 27 Sep 2023 15:42:26 +0000 (15:42 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Tue, 3 Oct 2023 12:51:13 +0000 (08:51 -0400)
commit56348379ec35bbccbb54e205cc180cc0bd4f9297
treeab202e25be5160e6546c850039e194b369c5dda3
parentf03cf6cc108022a05d036d4c197760203d31468b
Bug 34934: Remove the use of event attributes from OPAC lists page

This patch removes the use of event attributes (onclick, onchange) from
the OPAC lists template. These events are defined now along with the
other in-page JS.

The patch also removes a "delete list" button which was only shown
when viewing an empty list and which was redundant.

To test, apply the patch and log into the OPAC.

- Go to Lists -> New list.
- Change the category to "Private" and the "Allow changes" dropdown to
  "Staff only."
  - You should see a message, 'The "Staff only" permission has no actual
    effect while this list is strictly private'
- Change the category to "Public." The message should disappear.

- View the contents of a list.
  - Click the "Send list" button at the top of the table of titles.
  - It should trigger the "Sending your list" popup window.

Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-shelves.tt