Multiple fields widget should fully work in the contexual form

Created on 13 March 2025, 20 days ago

Overview

Not categorizing as *bug* because this, like many other things XB, is functionality that hasn't been built yet. Nothing has been broken, etc.

All the issues were found in the page data form. I assume this would hold true with the props for, but there aren't currently prop types that support offering a multi value widget.

Among other things, I could only add 2 items despite it being an unlimited item field, and clicking remove on an item resulted in the whole widget disappearing.

Proposed resolution

Obviously, get it working. Perhaps get it radix'd in the process.

User interface changes

πŸ“Œ Task
Status

Active

Version

0.0

Component

Semi-Coupled theme engine

Created by

πŸ‡ΊπŸ‡ΈUnited States bnjmnm Ann Arbor, MI

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • 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.

  • πŸ‡§πŸ‡ͺBelgium wim leers Ghent πŸ‡§πŸ‡ͺπŸ‡ͺπŸ‡Ί

    πŸ™ Music to my ears! πŸ˜„

  • Merge request !801#3512860 multi value widget support β†’ (Merged) created by bnjmnm
  • Pipeline finished with Failed
    14 days ago
    Total: 682s
    #452473
  • Pipeline finished with Failed
    14 days ago
    Total: 1544s
    #452486
  • Pipeline finished with Failed
    13 days ago
    Total: 1396s
    #453258
  • Pipeline finished with Failed
    13 days ago
    Total: 1552s
    #453396
  • Pipeline finished with Failed
    6 days ago
    Total: 1104s
    #459114
  • Pipeline finished with Failed
    6 days ago
    Total: 2116s
    #459138
  • Pipeline finished with Failed
    6 days ago
    Total: 1477s
    #459173
  • Pipeline finished with Failed
    6 days ago
    #459182
  • Pipeline finished with Failed
    6 days ago
    Total: 1402s
    #459206
  • Pipeline finished with Failed
    6 days ago
    Total: 2301s
    #459243
  • Pipeline finished with Canceled
    6 days ago
    Total: 207s
    #459267
  • Pipeline finished with Failed
    6 days ago
    Total: 2108s
    #459269
  • πŸ‡ΊπŸ‡ΈUnited States bnjmnm Ann Arbor, MI

    Feedback addressed

    • The media library e2e fail is unrelated and is addressed in this MR
    • The phpstan fail is unrelated and addressed in this MR
  • Pipeline finished with Failed
    5 days ago
    Total: 1449s
    #459860
  • Pipeline finished with Failed
    5 days ago
    Total: 1799s
    #459875
  • πŸ‡ΊπŸ‡ΈUnited States bnjmnm Ann Arbor, MI

    Review findings have been addressed. The CI reds are the same unrelated ones I called out in #6

    • The media library e2e fail is unrelated and is addressed in this MR
    • The phpstan fail is unrelated and addressed in this MR
  • πŸ‡¦πŸ‡ΊAustralia larowlan πŸ‡¦πŸ‡ΊπŸ.au GMT+10

    Adding parent Meta

  • Pipeline finished with Failed
    2 days ago
    Total: 1359s
    #461822
  • Pipeline finished with Failed
    2 days ago
    Total: 2820s
    #461831
  • πŸ‡ΊπŸ‡Έ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.

  • Pipeline finished with Failed
    about 9 hours ago
    #463341
  • πŸ‡ΊπŸ‡ΈUnited States bnjmnm Ann Arbor, MI

    The currently skipped-on-ci e2e tests are also passing

  • Pipeline finished with Canceled
    about 7 hours ago
    #463471
  • Pipeline finished with Running
    about 6 hours ago
    #463472
  • Pipeline finished with Failed
    about 5 hours ago
    #463566
  • Pipeline finished with Failed
    about 5 hours ago
    #463570
  • Pipeline finished with Failed
    about 5 hours ago
    #463538
  • Pipeline finished with Failed
    about 4 hours ago
    #463592
  • Pipeline finished with Canceled
    about 4 hours ago
    #463612
  • Pipeline finished with Failed
    about 4 hours ago
    #463614
  • Pipeline finished with Failed
    about 3 hours ago
    Total: 1484s
    #463650
  • Pipeline finished with Skipped
    about 2 hours ago
    #463687
    • bnjmnm β†’ committed 2e53da3a on 0.x
      Issue #3512860: Multi-valued fields' widget should fully work in the...
  • πŸ‡ΊπŸ‡Έ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! πŸ₯³

Production build 0.71.5 2024