From 88efe2571cadce2eddd132f4e99baf8ee3aa243b Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Mon, 16 Dec 2019 19:06:57 +0000 Subject: [PATCH] Bug 7882: (follow-up) Update sorting interaction This follow-up modifies the CSS for the sortable fields so that a different icon is used. Elsewhere in Koha (stock rotation stage sorting) we use the "bars" icon to indicate a sortable element. I think it would be good to continue to use this. The cursor is now styled as a "Move" cursor. The configuration of the sortable function in the script has been updated in two ways: 1. The movement of the element is constrained along the y-axis, and 2. The movement of the element is contstrained to the container within which it can be moved (for the tag, the tab, and for the subfield, the tag). To test, apply the patch and edit a MARC record in the basic cataloging editor. Confirm that the new icon is used, and that the "Move" cursor shows when you hover over the icon. Confirm that a tag can be dragged up and down within the constraints of the visible section. A subfield should be only movable within its parent tag. Signed-off-by: Maryse Simard --- koha-tmpl/intranet-tmpl/prog/css/addbiblio.css | 9 ++++++--- .../intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt | 10 ++++++++-- 2 files changed, 14 insertions(+), 5 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/css/addbiblio.css b/koha-tmpl/intranet-tmpl/prog/css/addbiblio.css index 28cf341..99ade19 100644 --- a/koha-tmpl/intranet-tmpl/prog/css/addbiblio.css +++ b/koha-tmpl/intranet-tmpl/prog/css/addbiblio.css @@ -26,16 +26,19 @@ ul li.tag, ul li.tag li.subfield_line { padding-left: 30px; } -ul li.tag:before, ul li.tag li.subfield_line:before { +ul li.tag::before, +ul li.tag li.subfield_line::before { + color: #999; + cursor: move; position: absolute; font-family: 'FontAwesome'; font-size: .8em; top: 0; left: 10px; - content: "\f047"; + content: "\f0c9"; } -ul li.tag:before { +ul li.tag::before { padding-top: 1.7em; } diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt index 08ab8ee..473556a 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt @@ -26,8 +26,14 @@ var $tabs = $('#addbibliotabs').tabs().bind('show.ui-tabs', function(e, ui) { $("#"+ui.panel.id+" input:eq(0)").focus(); }); - $( "ul.sortable_field", $tabs ).sortable(); - $( "ul.sortable_subfield", $tabs ).sortable(); + $( "ul.sortable_field", $tabs ).sortable({ + axis: "y", + containment: $("#addbibliotabs") + }); + $( "ul.sortable_subfield", $tabs ).sortable({ + axis: "y", + containment: "parent" + }); [% IF tab %] link = $("a[href='#[% tab | html %]']"); -- 2.7.4