All showing and hiding of #mkwsTermlists is now done directly on that
authorMike Taylor <mike@indexdata.com>
Thu, 4 Jul 2013 16:20:07 +0000 (17:20 +0100)
committerMike Taylor <mike@indexdata.com>
Thu, 4 Jul 2013 16:20:07 +0000 (17:20 +0100)
element rather than on its parent. Not only is this more correct, it
looks better because it avoids making the #mkwsRanking div leap across
to the right when the first results come in.

experiments/spclient/mkws.js

index e2e3ccf..84bb074 100644 (file)
@@ -160,12 +160,12 @@ function my_onstat(data) {
 function my_onterm(data) {
     // no facets
     if (!mkws_config.facets || mkws_config.facets.length == 0) {
-       $("#mkwsTermlists").parent().hide();
+       $("#mkwsTermlists").hide();
        return;
     }
 
     // display if we first got results
-    $("#mkwsTermlists").parent().show();
+    $("#mkwsTermlists").show();
 
     var acc = [];
     acc.push('<div class="title">' + M('Termlists') + '</div>');
@@ -602,7 +602,7 @@ function mkws_html_all(config) {
     domReady();
 
     // on first page, hide the termlist
-    $(document).ready(function() { $("#mkwsTermlists").parent().hide(); } );
+    $(document).ready(function() { $("#mkwsTermlists").hide(); } );
 }
 
 /* Responsive web design - change layout on the fly depending on
@@ -778,14 +778,14 @@ function mkws_mobile_resize () {
            $("#" + list[i]).hide();
        }
 
-       $("#mkwsTermlists").parent().hide();
-       obj = $("#mkwsTermlists").parent().html();
+       $("#mkwsTermlists").hide();
+       obj = $("#mkwsTermlists").html();
         $("#mkwsShiftedTermlists").html(obj);
     } else {
        for(var i = 0; i < list.length; i++) {
            $("#" + list[i]).show();
        }
-       $("#mkwsTermlists").parent().show();
+       $("#mkwsTermlists").show();
        $("#mkwsShiftedTermlists").html("");
     }
 };