Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

  • Cause: Vaadin TreeTable component normally supports partial updates if container is an ItemSetChangeNotifier (not sure why that condition).
    • But this is currently disabled with // FIXME: This disables partial updates until TreeTable is fixed so it does not change component hierarchy during paint
      • (warning) This needs to be clarified by the Vaadin guys, forcing Forcing partial updates seemed to work basically fine.
      • Vaadin commit said: Disable TreeTable partial updates until it is fixed. Currently TreeTable changes its child components during paint, which is too late for AbstractCommunicationManager to take into account
      • see http://dev.vaadin.com/ticket/8628
  • Effect: The whole table is repainted when expanding one node.
    • This is not the most harmful factor for user interaction, but it surely has its share of impact when many nodes are expanded.
  • (info) Inplace editing is not yet using partial updates, although that was also done in the standalone PoC (ticket already exists).
  • Fix: Patch TreeTable#setContainerDataSource() method to set containerSupportsPartialUpdates = true.

...