Drag-and-drop reordering doesn't work if the region content is nested in other elements

Is this something that could be tested with a functional JS test?

This is great, thank you @pinkdexo
It helps me to save the use of the layout of the TheMag theme -a great work-.
In my case Drupal 8.8.4 I used the #4 patch.
Success to get back the useful drag and drop !

As Patch is created for above issue i changed the status need work to need review.

This patch (tested the 8.9.x) works flawlessly for me.

Patch #10 works for me, too.

Glad it's working! Now just for the automated test coverage...

I am working on this

check the drupal standards

Re-rolled patch #18, Attached interdiff for same.

Fixed custom command failure

I tried writing tests for this and boy was it hard! I have a partial test in this patch. I don't know how to make it any better.

The problems I had were:

1. SortableTestTrait::sortableTo() only simulates a drag and does actually have the browser perform drag and drop.
2. For some reason I don't understand, NodeElement::dragTo() wasn't working for LB blocks, even with the default templates.

This is just a new fail test, so no interdiff.

Sorry...Here's another.

That failed exactly as expected. Clicking the body block did not trigger any sort of "sortable" behavior. Here's the fail patch with the fix from #10 added.

NOTE: Ignore that interdiff. The correct one is attached to #26.

Ignore that interdiff. Oof.

Here's the real interdiff, which is simply the fix from #10.

Aha! Here's why my attempts to use NodeElement::dragTo() weren't working. Per the SortableTestTrait:

* Selenium uses ChromeDriver for FunctionalJavascript tests, but it does not
* currently support HTML5 drag and drop. These methods manipulate the DOM.
* This trait should be deprecated when the Chromium bug is fixed.

This is tracked in an issue: #3078152: Follow-up to #3064049 Deprecate sortable Trait.

I'm pretty sure we can't make a good test for this issue until Chromium supports html5 drag and drop. I say that because the following test runs flawlessly on 9.3.x.

/**
   * Tests the Layout Builder UI with extra wrapper divs in template.
   */
  public function testLayoutBuilderCustomTemplate() {
    // Override the layout templates.
    \Drupal::service('module_installer')->install(['layout_builder_custom_template_test']);
    \Drupal::service('plugin.manager.core.layout')->clearCachedDefinitions();
    \Drupal::service('theme_handler')->refreshInfo();
    $this->testLayoutBuilderUi();
  }

That is, big existing test does not fail with the custom templates featuring extra wrapping divs.

I think the wisest move would be to postpone this issue until the Chromedriver bug is fixed. To me it looks like there's no existing robust test coverage of LB drag-and-drop, which makes changes really risky!

That said, anyone running into this bug will probably be ok with the patch in #10.

The fail patch from #23 will be easy to update when we can do real dragging, so that's a good start.

Thank you thank you thank you!

Patch 25 works for me just fine! I can RTBC!

This still needs tests, so back to NW.

We are seeing this same bug with Drupal core 9.4.8 and following contrib modules:
- Field group 8.x-3.3
- UI Patterns 8.x-1.3.

We have a content display mode with multiple regions

Regions are something like:
- Header
- Main content
- Footer

When I create a field group inside the region, the drag breaks completely.

There is also a related issue in Field group: https://www.drupal.org/project/field_group/issues/3085858.

I have tried the patches from issues 3089151, 3087608 and this one. It seems it gets a little bit better and with trial and error I can get the field to stay in a correct region. But sometimes it just resets. Here is a attached video recording of what happens when I drag multiple times and then save.

Without the patches, the dragging is completely broken and you cannot even stop the drag. The field you are dragging just keeps following you. With the patch from issue 3087608 the dragging works a bit better but it let's me let go of dragging. But after saving the form, the region is still reset.

Without the patches the functionality seemed to be that if I try to drag the field to a region, it gets assigned to the first available region instead of the one I would like to drag the field to. I have tested this with Claro, Gin and Seven admin themes.

Rerolls the patch to make it compatible with 9.5

Looked like the reroll was missing the tests that were in the patch from #25 so pushed those to both the 9.5 and 10 MRs

Fixed for drupal/core:9.5.3

Patch rerolled for 9.5.x
Added reroll diff.

Was previously tagged for tests which still need to happen.

Also issue summary could use some updating.

Thanks.

Rerolled for D10.1.x

I can confirm #44 patch applies to D10.1.8

@weseze - Please provide interdiff
Also we need a re-roll for 11.x