vaadin/flow 9.0.23

Vaadin Flow 9.0.23

on GitHub

Changes since 9.0.22

All changes

Fixes

• Added HasDynamicTitle checking in parent layouts (#15355)
  Commit · Pull request · Issue

  Checks the navigation targets including parent layouts and takes the page title from the one implementing HasDynamicTitle.

• Do not remove expanded items from key mapper (#15314)
  Commit · Pull request · Issue

  With a TreeGrid, the expanded state of a root item gets lost when scrolled enough to request a new range. The cause for this is the updated key of the returned item. The expanded state for the items except the root items do not get lost. This PR makes sure that the key-value pair for a root item persists in the KeyMapper if the item is expanded. This change only affects hierarchical data. A test is added to ensure that the fix solves the issue. While this refactoring is less than ideal in terms of clean code, I think that the alternative requires a much larger refactoring. The logic in the regular and hierarchical data communicators are mostly both coupled and duplicated with private methods at this time. Therefore, this PR is as atomic as possible.

• Remove unnecessary clear condition (#15282)
  Commit · Pull request · Issue

  For hierarchical data, even when the child components are filtered out, the clear method on ArrayUpdater.Update in HierarchicalCommunicationController is never called. This is caused by an if check, in which the start index of previousActive is compared to the newly calculated assumedSize. Since there might be items to clear, the newly calculated size should have no effect on whether we clear the items or not. This check is not present in the DataCommunicator counterpart within the method collectChangesToSend. This PR removes the aforementioned clear condition. Add test to ensure that the expanded children are cleared properly when filtered out.