Bugzilla – Attachment 90516 Details for
Bug 23099
OPAC Search result sorting "go" button flashes on page load
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 23099: OPAC Search result sorting "go" button flashes on page load
Bug-23099-OPAC-Search-result-sorting-go-button-fla.patch (text/plain), 3.38 KB, created by
Owen Leonard
on 2019-06-11 18:49:29 UTC
(
hide
)
Description:
Bug 23099: OPAC Search result sorting "go" button flashes on page load
Filename:
MIME Type:
Creator:
Owen Leonard
Created:
2019-06-11 18:49:29 UTC
Size:
3.38 KB
patch
obsolete
>From 5a1b6f4ec3a25ccad13395fb1fd6bbd04e77d079 Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Tue, 11 Jun 2019 18:13:08 +0000 >Subject: [PATCH] Bug 23099: OPAC Search result sorting "go" button flashes on > page load > >There are a few places in the OPAC where we hide the submit button on a >sorting form because the form auto-submits when the selection changes. >These should be hidden using CSS instead of JS to prevent the temporary >appearance of the submit button while the page loads. > >To test, apply the patch and rebuild the OPAC CSS >(https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client). > >Test OPAC search results and OPAC checkout history. On both pages you >should not see a "Go" button next to the <select> input for changing the >sort order. > >Disable JavaScript in your browser and test those pages again. The "Go" >button should appear. >--- > koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss | 8 ++++++++ > koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-readingrecord.tt | 1 - > koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results-grouped.tt | 1 - > koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt | 1 - > 4 files changed, 8 insertions(+), 3 deletions(-) > >diff --git a/koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss b/koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss >index 29fc16c..78cacf4 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss >+++ b/koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss >@@ -17,12 +17,20 @@ $footer-height: 45px; > .selections-toolbar { > display: none; > } >+ >+ #sortsubmit { >+ display: inline; >+ } > } > > .js { > .dateformat { > display: none; > } >+ >+ #sortsubmit { >+ display: none; >+ } > } > > .fa.fa-icon-black, >diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-readingrecord.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-readingrecord.tt >index 5a880b2..86f92fc 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-readingrecord.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-readingrecord.tt >@@ -210,7 +210,6 @@ > <script> > $(document).ready(function(){ > [% IF ( GoogleJackets ) %]KOHA.Google.GetCoverFromIsbn();[% END %] >- $('#sortsubmit').hide(); > $('#order').change(function() { > $('#sortform').submit(); > }); >diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results-grouped.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results-grouped.tt >index 31194b9..8e4ac21 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results-grouped.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results-grouped.tt >@@ -297,7 +297,6 @@ $(document).ready(function(){ > }); > [% END %] > >- $('#sortsubmit').hide(); > $('#sort_by').change(function() { > $('#bookbag_form').submit(); > }); >diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt >index e6bd51b..e15842d 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt >@@ -828,7 +828,6 @@ $(document).ready(function(){ > param1 += "<\/select> <input type=\"submit\" class=\"btn btn-small\" value=\""+_("Save")+"\" />"; > [% END %] > >- $('#sortsubmit').hide(); > $('.resort').change(function() { > $('#bookbag_form').submit(); > }); >-- >2.1.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 23099
:
90516
|
91723
|
91865