[Bug Report][3.3.14] VList with selection doesn't work when children is an empty array

[mlankamp]

Environment

Vuetify Version: 3.3.14
Vue Version: 3.3.4
Browsers: Edge 117.0.2045.9
OS: Windows

Steps to reproduce

Using the prepend template the checkbox is added with the needed events. When the children property is set to an empty array, the select functionality doesn't work. When the children is set to null or undefined it does work

Expected Behavior

Allow the children to be set to empty array.

Actual Behavior

An empty array causes the children to behave differently

Reproduction Link

https://play.vuetifyjs.com/#...

[mlankamp]

The empty array causes the children to render differently:

[mlankamp]

The problem can be solved in VListChildren.tsx (

vuetify/packages/vuetify/src/components/VList/VListChildren.tsx

Line 66 in 26aa5b1

return children ? (

)

to add an additional check (besides null and undefined) for the array length
return children && children.length > 0 ? (

This solves the problem, and when there are no children it's more logical to render a listitem and not a listgroup

[KaelWD]

I believe this is intentional so you can load children on-demand: https://play.vuetifyjs.com/#eNqtVktv3DYQ/iuEcvAaWEkp0l4W3iBBm0N7KpC2F69R0NLIYkqRAkmtkRr73zt8raiHs24SXSQNZ75vHuQMb5+y931fHAfIdtmNga7n1MDbgyDk5pjTvnef7qeSwlAmQAWRE3KmDTnmnayB72QPAur9IbMfh4zsGAJq/HfvQ3Y2RNNIRV61QGtQqPVEeiV7TU4TzTNNblGQ654Jy+F0Z5op7jFnDapxSWsmHoqW6o0zKY6UD3CN7r3C8EDUMwTHh5oPCrTOK6aqgVNFkBQMqI4JC67Zv4DguqOcI1I596KcZHIUJ5GkyViqR9WYfvyd5t9KfHkm1virK8V6QzSYwVePdb1UhjyRSnb9YKDeEgUNOZFGyY5cYe2vrCHBdYHVdMUie6uzufX0T9EtwwyHHbn6U4PSV9sodilF8TAVVy3jtQKxI7d3QXhy77vrlDCUKFAKeCQfwWyuJzp/2y0VnbLmcSHIY2ib4OqDRRj9VpgMJTyK3wCpO8Qma4PixMKjozN/UY74t3dJrEUj1QdatdZkALJ/O5phthuySXgKJio+1KC98nVCkTjmeYp+0AHUReif0/jpncKNOXCDTtWN3rhqeaqtr0Nian3x2kWsRcFBPJh25sYX+QlwDVP9eKp8iLSuF1aENGCq9udAG9YL0+J39GWWOfukmVv1ZsmOnQdP5qriLHa7T8JnqndaS/YoTFxCAJ+pdPdYTftuBlEZJsVYlliQGCRuG7IZDxmRjT9sSTFsxaws8u33M4xk01i9s7vehwlEjHVi+8wWOit/aRNdR2b/m3CfX8EPz4Ld4Ajq19BPvlsvGV2bbJ6I977GJj0apph0ubSKu8BO8X+TrSC/SJjipDSfUCOvlxqx21ymoAIuUKDGZYrYdecrq3n7UDMj1XriYGXtKzP3sWOmvZA7vabz/7J3icbm7wU06xmcTTO/56n+LKqxC6y1v+gzfaTMDRfyO45gpsEeMMmPbpbgLPqDdSAHE6Vb8tPr13YePnOAk3M2Pb4r/X85FOcDoqO9awkrc+35tnJGm1WlKAprNMvxysVgMevWxs5K41tp2XPLOGMHvKtNOtVN6e9KeEvKtpm/JuUYfvFJS4H3YYeAt1e3gHfNXcQ8ZHhpsv+HrDWm17uyrGqBZjiL2FEVAkwp+q58h2qlGoTBeuJp7d69Kd4UP5Y1XuxScQG6y++VfMRCIsghCzlxNIY1n19AFTQdww+BIshyTu+15Vhgl37n5FgKvIKDemlIM7M0rNnSIrSQ+hMmnDPxj57lutIuz6GjfGvMDi0c0VN22mbOUaT2Ftndf0L/3NI=

v2 treeview did this too if load-children was set.

[mlankamp]

I will check if this conflict with my proposed change. If so, I will close this PR

[mlankamp]

Closed the PR, doesn't work for an async scenario