Bug 15119: Hide search header text boxes on render

When the page rendering is slow or the page is heavy, the tabs in the
header shown be not correctly displayed as tabs.

This patch suggests to hide them, display the first one, and wait for
the JS code to display them nicely.

To easily reproduced the ergonomic issue, go on the circulation home
page (/cgi-bin/koha/circ/circulation.pl) and search for a patron 'a' or
'd', you will get a lot of result and the page will be slow to be fully
displayed.

Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com>
Works as advertised

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Brendan Gallagher brendan@bywatersolutions.com
This commit is contained in:
Jonathan Druart 2016-01-06 13:10:00 +00:00 committed by Brendan Gallagher
parent b72f9b8c51
commit 70eedf2217
2 changed files with 11 additions and 0 deletions

View File

@ -2286,6 +2286,15 @@ ul.ui-tabs-nav li {
border-radius: 0 0 4px 4px;
}
#header_search > div,
#header_search > ul > li {
display: none;
}
#header_search > div:first-of-type,
#header_search > ul > li:first-of-type {
display: block;
}
.authref {
font-style: normal;
text-indent: 4em;

View File

@ -34,6 +34,8 @@ $.fn.selectTabByID = function (tabID) {
if($("#header_search #catalog_search").length > 0){ shortcut.add('Alt+q',function (){ $("#header_search").selectTabByID("#catalog_search"); $("#search-form").focus(); }); } else { shortcut.add('Alt+q',function(){ location.href="/cgi-bin/koha/catalogue/search.pl"; }); }
if($("#header_search #renew_search").length > 0){ shortcut.add('Alt+y',function (){ $("#header_search").selectTabByID("#renew_search"); $("#ren_barcode").focus(); }); } else { shortcut.add('Alt+y',function(){ location.href="/cgi-bin/koha/circ/renew.pl"; }); }
$("#header_search > ul > li").show();
$(".focus").focus();
$(".validated").each(function() {
$(this).validate();