Attachment 77706 Details for Bug 19390 – Bug 19390: Make jQuery selector more specific, so OPAC view link can open in new tab

[patch] Bug 19390: Make jQuery selector more specific, so OPAC view link can open in new tab

Bug-19390-Make-jQuery-selector-more-specific-so-OP.patch (text/plain), 2.12 KB, created by Katrin Fischer on 2018-08-10 17:47:53 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2018-08-10 17:47:53 UTC

Size: 2.12 KB

patch

obsolete

>From 36a435e09dca4d7a1445135240e72ee5809e8f8c Mon Sep 17 00:00:00 2001
>From: Katrin Fischer <katrin.fischer.83@web.de>
>Date: Fri, 10 Aug 2018 17:18:39 +0000
>Subject: [PATCH] Bug 19390: Make jQuery selector more specific, so OPAC view
> link can open in new tab
>
>The OPAC view link in the staff result list already had a target="_blank",
>but it didn't work, because of the JavaScript for the result list browser
>in staff.
>
>The JavaSript code was looking for the links to the detail page in staff
>and this also selected the link to the detail page in OPAC. By changing
>detail.pl to \detail.pl opac-detail.pl will no longer be selected.
>
>To test:
>- Search in the staff interface
>- Click "OPAC view" links in staff result lists
>- Click "OPAC view' links in detai page
>- Verify both now open in a new tab
>- Click other links and test that navigation (previous, next,
>  return to results) works as expected
>- Inside the staff client, you should see something like
>  searchid=scs_1533922927978 added to the URLs
>---
> koha-tmpl/intranet-tmpl/js/browser.js | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/js/browser.js b/koha-tmpl/intranet-tmpl/js/browser.js
>index 7e40785..173eaa0 100644
>--- a/koha-tmpl/intranet-tmpl/js/browser.js
>+++ b/koha-tmpl/intranet-tmpl/js/browser.js
>@@ -63,10 +63,10 @@ KOHA.browser = function (searchid, biblionumber) {
>         sessionStorage.setItem('searches', JSON.stringify(searches));
>         $(document).ready(function () {
>             //FIXME It's not a good idea to modify the click events
>-            $('#searchresults table tr a[href*="detail.pl"]').on('click', function (ev) {
>+            $('#searchresults table tr a[href*="/detail.pl"]').on('click', function (ev) {
>                 ev.preventDefault();
>             });
>-            $('#searchresults table tr a[href*="detail.pl"]').on('mousedown', function (ev) {
>+            $('#searchresults table tr a[href*="/detail.pl"]').on('mousedown', function (ev) {
>                 if ( ev.which == 2 || ev.which == 1 && ev.ctrlKey ) {
>                     // Middle click or ctrl + click
>                     ev.preventDefault();
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 19390: 77706 | 77713 | 77738