From de39fc6e94c86945d59e1eb4a8e9d35f244dad0f Mon Sep 17 00:00:00 2001 From: Andreas Roussos Date: Sat, 6 Oct 2018 10:15:26 +0300 Subject: [PATCH] Bug 21506: DataTables four button pagination uses the wrong icon for First and Last buttons MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Pages with DataTables' four button pagination use the wrong icon for 'First' and 'Last'. Also, the icons for all four buttons are truncated at the bottom. This patch fixes that. Test plan: 1) Go to a page that uses four button pagination (e.g. Home > Administration > MARC frameworks) and notice how the pagination icons for First/Previous and Next/Last are the same. Also notice all four icons are clipped at the bottom. 2) Apply the patch. 3) Refresh the page; the First and Last buttons' icons now include a vertical bar, and there is no clipping visible. 4) For completeness, check other affected pages as well. Signed-off-by: Séverine QUEUNE --- koha-tmpl/intranet-tmpl/prog/css/datatables.css | 4 +++- koha-tmpl/intranet-tmpl/prog/js/datatables.js | 8 ++++---- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/css/datatables.css b/koha-tmpl/intranet-tmpl/prog/css/datatables.css index 71fdb6b..94b03e8 100644 --- a/koha-tmpl/intranet-tmpl/prog/css/datatables.css +++ b/koha-tmpl/intranet-tmpl/prog/css/datatables.css @@ -167,7 +167,7 @@ div.dataTables_paginate.paging_four_button { .paginate_enabled_last { cursor: pointer; *cursor: hand; - padding: .1em 0; + padding: .2em 0; } .paginate_disabled_previous, @@ -177,6 +177,8 @@ div.dataTables_paginate.paging_four_button { color: #111 !important; } +.paginate_disabled_first, +.paginate_enabled_first, .paginate_disabled_previous, .paginate_enabled_previous { padding-left: 23px; diff --git a/koha-tmpl/intranet-tmpl/prog/js/datatables.js b/koha-tmpl/intranet-tmpl/prog/js/datatables.js index f8d23b9..0e95d1e 100644 --- a/koha-tmpl/intranet-tmpl/prog/js/datatables.js +++ b/koha-tmpl/intranet-tmpl/prog/js/datatables.js @@ -263,24 +263,24 @@ $.fn.dataTableExt.oPagination.four_button = { var buttons = an[i].getElementsByTagName('span'); if ( oSettings._iDisplayStart === 0 ) { - buttons[0].className = "paginate_disabled_previous"; + buttons[0].className = "paginate_disabled_first"; buttons[1].className = "paginate_disabled_previous"; } else { - buttons[0].className = "paginate_enabled_previous"; + buttons[0].className = "paginate_enabled_first"; buttons[1].className = "paginate_enabled_previous"; } if ( oSettings.fnDisplayEnd() == oSettings.fnRecordsDisplay() ) { buttons[2].className = "paginate_disabled_next"; - buttons[3].className = "paginate_disabled_next"; + buttons[3].className = "paginate_disabled_last"; } else { buttons[2].className = "paginate_enabled_next"; - buttons[3].className = "paginate_enabled_next"; + buttons[3].className = "paginate_enabled_last"; } } } -- 2.1.4