sylvainjule / kirby-multilist Goto Github PK

Hey there,
I don't know about prior Kirby versions, but editing nested 'multilist' entries isn't possible in v3.7. Thanks for your library!

Cheers,
S1SYPHOS

Hey there,
I just tried this out and it works great! One thing though was the column width seems off:

When assigning width to all properties, it looks like this:

Total width is alright (as expected), but table styles are still off 😕

Cheers
S1SYPHOS

Hey Sylvain,

great field, thank you - and just at the moment I was interested in such field!

While working with it, I quickly grew frustrated with few things however - namely:

• I can't add a new row with keyboard only
• If I want to change a thing X rows up or down, I have to tab or shift-tab however many times it's needed, even though the other cell is straight up or whatever

Would you be interested in accepting PR for keyboard shortcuts? I am not sure how hard it can be, but sometime next week I could try to add them. If you would accept it, what would the shortcuts be? I find myself reaching for ⌘ + arrows and ⌘ + enter for new row, but would also accept any other option.

Hey Sylvain! Thanks a lot for this great plugin. It's an essential to work fast with list content.
I just tried it for the first time with Kirby 3.6 and unfortunately it's UI is broken:

Kirby 3.5.x:

Kirby 3.6.0:

The table headers are very small, the drag-and-drop area is gone and the remove-button on the right is moved outside the field. I investigated and found the reason: the 38px height and width of all those elements was refactored into the variable --item-height: 38px. This variable is set on .k-structure-field.

I fixed it locally by adding it to .k-multilist-table:

.k-multilist-table {
    --item-height: 38px;
}

Hey there,
it would be great to easily duplicate entries (which makes repeatedly adding entries way faster), I know feature requests are always on-top stuff, but maybe something to consider. Otherwise close it, anyway thanks for this plugin!

Cheers
S1SYPHOS

The field works - it exists, can be changed, and is at least readonly, but it's still showing. Will you accept PR for hiding field of type hidden?

(or, can you add it please?)

If you have the field set as sortable: false and sortBy: whatever however, as soon as you change the value of the field the multilist is sorted by, if the order of the data changes, the focus doesn't.

• enter value at position 3
• change the sortBy field to whatever resorts the content
• data changes (your new field is now row 1)
• focus stays on row 3

I updated my (still unmerged 😔 - what's up @sylvainjule) 3.8.0 compatibility pull request and added v4 compatibility adjustments: #12

Before:

After: