fix: do not render hidden combo box items

[sissbruecker]

Since adding the data provider controller to combo box in #7044, the component now calls ComboBoxMixin.requestContentUpdate when filtering using a data provider. That method iterates over every item in the overlay, regardless whether visible or hidden, and runs its renderer function:

web-components/packages/combo-box/src/vaadin-combo-box-mixin.js

Lines 412 to 414 in d877284

	this._getItemElements().forEach((item) => {
	item.requestContentUpdate();
	});

That can lead to an issue in the Flow component when using component renderers:

• Entering a filter can move an item's content element to a different item (for example the first one if there is only one match)
• The item that originally contained the content element is then hidden and not updated - it still references the same data, and thus the same content element
• Then we run the renderer function for the hidden item, and move the content element back to it
• The result is that the visible item ends up being empty

This change fixes the issue by not running the renderer function for hidden items.

Fixes vaadin/flow-components#6810

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[vaadin-bot]

This ticket/PR has been released with Vaadin 24.6.0.beta1 and is also targeting the upcoming stable 24.6.0 version.