The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- First commit to issue fork.
- @arunkumark opened merge request.
- Status changed to Needs review
almost 2 years ago 8:40am 24 March 2023 - 🇮🇳India arunkumark Coimbatore
I have re-rolled the patch and created the MR
- Status changed to Needs work
almost 2 years ago 10:07pm 25 March 2023 - 🇺🇸United States smustgrave
Be useful to have step to reproduce the issue.
Will need a test case showing the issue also.
- Status changed to Needs review
about 1 year ago 1:47am 6 December 2023 - 🇻🇳Vietnam linhnm
I encountered this issue on Drupal 10.1.
Steps to reproduce
-
Create a custom field widget with tabledrag for unlimited fields..
$element['table-row'] = [ '#type' => 'table', '#header' => [ $this->t('Column @number', [ '@number' => $delta + 1, ]), $this->t('Weight'), $this->t('Parent'), ], '#empty' => $this->t('Sorry, There are no items!'), '#prefix' => "<div id='links-fieldset-wrapper-{$delta}'>", '#suffix' => '</div>', '#tabledrag' => [ [ 'action' => 'match', 'relationship' => 'parent', 'group' => 'row-pid', 'source' => 'row-id', 'hidden' => TRUE, 'limit' => FALSE, ], [ 'action' => 'order', 'relationship' => 'sibling', 'group' => 'row-weight', ], ], ];
-
The tabledrag icon for the multiple fields is being added to the last child js-indentation
I have re-rolled the patch for the 10.1
-
Create a custom field widget with tabledrag for unlimited fields..
- Status changed to Needs work
about 1 year ago 1:56am 6 December 2023 - 🇺🇸United States smustgrave
Was tagged for tests which are still needed.
Probably needs an issue summary update also using the standard issue template