progressbar script change

The progressbar script used in the search accessed parts of the DOM before
it was completely parsed which caused problems in IE. This patch moves back
to document.write for outputting the image.

darcs-hash:20060924170354-7ad00-1e0247b00fc4da4dd30301f4b9389ef727496c4a.gz

2 files changed, 12 insertions, 3 deletions

diff --git a/inc/html.php b/inc/html.php
index 46e6bd8c6..dd2aa2ca0 100644
--- a/inc/html.php
+++ b/inc/html.php
@@ -363,10 +363,10 @@ function html_search(){
 
   //show progressbar
   print '<div class="centeralign" id="dw__loading">';
-  print '<br /></div>';
   print '<script type="text/javascript" charset="utf-8">';
-  print 'showLoadBar("dw__loading");';
+  print 'showLoadBar();';
   print '</script>';
+  print "<br /></div>\n";
   flush();
 
   //do quick pagesearch
diff --git a/lib/scripts/script.js b/lib/scripts/script.js
index 8c0485671..c917c365a 100644
--- a/lib/scripts/script.js
+++ b/lib/scripts/script.js
@@ -188,9 +188,17 @@ function prependChild(parent,element) {
 /**
  * Prints a animated gif to show the search is performed
  *
+ * Because we need to modify the DOM here before the document is loaded
+ * and parsed completely we have to rely on document.write()
+ *
  * @author Andreas Gohr <andi@splitbrain.org>
  */
-function showLoadBar(id){
+function showLoadBar(){
+
+  document.write('<img src="'+DOKU_BASE+'lib/images/loading.gif" '+
+                 width="150" height="12" alt="..." />');
+
+  /* this does not work reliable in IE
   obj = $(id);
 
   if(obj){
@@ -198,6 +206,7 @@ function showLoadBar(id){
                     'width="150" height="12" alt="..." />';
     obj.style.display="block";
   }
+  */
 }
 
 /**