vuetify icon indicating copy to clipboard operation
vuetify copied to clipboard

[Documentation] VList example is incomplete

Open PaulCombal opened this issue 9 months ago • 2 comments

Problem to solve

I want to make a list of checkboxes using lists. Naturally I go to lists. There is an example of how to do so but it's incomplete. https://vuetifyjs.com/en/components/lists/#action-and-item-groups

Why do they put a "value" prop on the list-item if it's to never be used? How do I get that value back?

Proposed solution

The v-list tag in this example should be:

<v-list lines="three" select-strategy="classic" v-model:selected="slectedSettings">

To show that the values are actually stored in selectedSettings.

PaulCombal avatar May 16 '24 17:05 PaulCombal

I guess the value prop in v-list-item is to define the assigned value to v-list's v-model:selected when the item is selected.

Just yesterday I proposed the docs change to v-list-item because I was confused too. https://github.com/vuetifyjs/vuetify/pull/19940/files

t407o avatar Jun 04 '24 14:06 t407o

It is incorrectly ported from v2 and should actually ~~be hidden in my opinion~~. (edit: it is currently the only example with v-list-item-action on this page, so should be fixed and left there)

  • missing script is the obvious part
  • markup does not include v-list-group although description says it does
  • does not showcase selection-strategy="classic" but rather leaf
  • isActive can be replaced with isSelected which is confusing

The problem is v-list-group no longer emits nor binds any model for selection state. And the actual example for v-list-group with classic selection has this weird behavior I cannot pinpoint to specific line in the sources...

If I understand the direction of the changes, groups are now only for expand/collapse and it is already pretty clearly presented in the "Sub group" example (if we ignore huge indents and kinda unexpected collapsing.. I wish it had open-strategy="multiple" instead of default one).

To be frank I don't understand the "easily connect actions to your tiles" even in the v2 version, so I wish I could get an opinion from someone from the core team.

J-Sek avatar Aug 22 '24 09:08 J-Sek