Fix issue #7058: WTableView displays "Loading" when resized...

Adapted from a patch provided by Bruce Toll. I
chose to go with putting the initial scroll functionality into the
wtResize of WTableView.js, rather than splitting up wtResize.

Author: RockinRoel

src/Wt/WTableView.C

@@ -791,13 +791,16 @@ void WTableView::defineJavaScript()
 
   LOAD_JAVASCRIPT(app, "js/WTableView.js", "WTableView", wtjs1);
 
-  setJavaScriptMember(" WTableView", "new " WT_CLASS ".WTableView("
-		      + app->javaScriptClass() + "," + jsRef() + ","
-		      + contentsContainer_->jsRef() + ","
-		      + headerContainer_->jsRef() + ","
-		      + headerColumnsContainer_->jsRef() + ",'"
-		      + WApplication::instance()->theme()->activeClass()
-		      + "');");
+  WStringStream s;
+  s << "new " WT_CLASS ".WTableView("
+    << app->javaScriptClass() << ',' << jsRef() << ','
+    << contentsContainer_->jsRef() << ','
+    << viewportTop_ << ','
+    << headerContainer_->jsRef() << ','
+    << headerColumnsContainer_->jsRef() << ",'"
+    << WApplication::instance()->theme()->activeClass()
+    << "');";
+  setJavaScriptMember(" WTableView", s.str());
 
   if (!dropEvent_.isConnected())
     dropEvent_.connect(this, &WTableView::onDropEvent);
@@ -813,18 +816,6 @@ void WTableView::defineJavaScript()
     columnResizeConnected_ = true;
   }
 
-  if (viewportTop_ != 0) {
-    WStringStream s;
-    s << "function(o, w, h) {"
-      <<   "if (!o.scrollTopSet) {"
-      <<     "o.scrollTop = " << viewportTop_ << ";"
-      <<     "o.onscroll();"
-      <<     "o.scrollTopSet = true;"
-      <<   "}"
-      << "}";
-    contentsContainer_->setJavaScriptMember(WT_RESIZE_JS, s.str());
-  }
-
   if (canvas_) {
     app->addAutoJavaScript
       ("{var obj = " + jsRef() + ";"

src/js/WTableView.js

@@ -8,7 +8,7 @@
 
 WT_DECLARE_WT_MEMBER
 (1, JavaScriptConstructor, "WTableView",
- function(APP, el, contentsContainer, headerContainer, headerColumnsContainer,
+ function(APP, el, contentsContainer, initialScrollTop, headerContainer, headerColumnsContainer,
       selectedClass) {
    el.wtObj = this;
 
@@ -33,6 +33,7 @@ WT_DECLARE_WT_MEMBER
 
    var scrollX1 = 0, scrollX2 = 0, scrollY1 = 0, scrollY2 = 0;
    var scrollToPending = false;
+   var initialScrollTopSet = initialScrollTop === 0;
 
    /*
     * We need to remember this for when going through a hide()
@@ -66,6 +67,11 @@ WT_DECLARE_WT_MEMBER
    };
 
    contentsContainer.wtResize = function(o, w, h, setSize) {
+     if (!initialScrollTopSet) {
+       o.scrollTop = initialScrollTop;
+       o.onscroll();
+       initialScrollTopSet = true;
+     }
      if ((w - currentWidth) > (scrollX2 - scrollX1)/2 ||
          (h - currentHeight) > (scrollY2 - scrollY1)/2) {
        currentWidth = w; currentHeight = h;