Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.
Comment: latest findings on scroll jumps

...

  • Effect: The tree might still jump slightly when expanding/collapsing nodes, just for the time it repaints.
  • Cause: There It is likely that there is some scroll position hack in TreeTableConnector connectors that we should assert how useful it is.
    • FOUND! line 138 in TableConnector: getWidget().updateFirstVisibleAndScrollIfNeeded(uidl);
    • It is even documented in VScrollTable as /** For internal use only. May be removed or replaced in the future. */
  • Fix: If experiment is convincing, this is also a matter of patching that Vaadin class Patch TableConnector to remove the call or VScrollTable to remove the body of that method.

 

Disculped

  • It is interesting to note that our JCR containers are fine; they are not causing performance flaws.
    • Some methods however should be left unsupported since the container should be read-only, in other words, persistance is not managed by the container.
    • There is interesting insight on this in the Vaadin book section about SQL container.

...