- Issue created by @bnjmnm
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
I ran into this months ago in #3467870 β see #3467870-16: Support `{type: array, β¦}` prop shapes β , which included this screenshot:
(back in September!)
- πΊπΈUnited States bnjmnm Ann Arbor, MI
Re #2. So π [later phase] Support matching `{type: array, β¦}` prop shapes Postponed should definitely be blocked on this. We'll scope this at the widget JS functionality (controls show up, it is sortable, 'add' and 'remove' add and remove the correct things, etc), which should free up #3467870 to focus on what is attempting to be built there.
- πΊπΈUnited States bnjmnm Ann Arbor, MI
- πΊπΈUnited States bnjmnm Ann Arbor, MI
- πΊπΈUnited States bnjmnm Ann Arbor, MI
Knew the tabledrag targeting in
ajax.hyperscriptify.js
wasn't great - adding a few attributes per @larowlan suggestion does a much better job of it. - πΊπΈUnited States bnjmnm Ann Arbor, MI
The currently skipped-on-ci e2e tests are also passing
- Merge request !834Draft: #3512860 try a few things to see if Cypress chills out β (Open) created by bnjmnm
-
bnjmnm β
committed 2e53da3a on 0.x
Issue #3512860: Multi-valued fields' widget should fully work in the...
-
bnjmnm β
committed 2e53da3a on 0.x
- πΊπΈUnited States bnjmnm Ann Arbor, MI
Given the required approvals are made, this passes tests, and there's several other issues that will benefit from the hyperscriptify and test improvements, I'm going to merge this in.
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Great to see this starting to work, this unblocks initial
tyoe: array
support as indicated in #2! π₯³