Missing div closing tag breaks block config layout

The code change is really simple. See MR.

Because there is no difference between last 2.0.x and and current 4.0.x I just changed the target branch to 4.0.x in MR and for this issue.

I don't want to get this fix messed up. But just a note to improve translations. There is currently there is lot's of unneeded html inside the t():

$help_text .= $this->t('<a href="@url_used_attribute_list@" class="use-ajax" data-dialog-options="{&quot;width&quot;:800}" data-dialog-type="modal">See all the attributes used</a>.', [
          '@url_used_attribute_list@' => $url_used_attribute_list,
        ]);

It would be better to only put "See all the attributes used" inside t().

Hi!

The patch works as expected for the attribute fields group:

But it does nothing for the class fields group:

Note: translation of "Add another class" and "Remove last added class" could be included as well.

Thanks for your feedback @zigazou

translation of "Add another class" and "Remove last added class" could be included as well

This one has been fixed at 🐛 Some button labels are not translatable Active

@rentog, Yes, you are right here. I expect the same thing to be pointed out here for the multilingual sites.

Thanks for this, In my case, it was hiding the save and delete button were being hidden. Which was bizzare,

The patch fixed it in the new Drupal CMS V1

The status "Needs work" was argued to translations which are not focus of this issue and were fixed as reported at #8 and #9. So switching back to "Needs review".

Thanks @thomas kaisuka for testing. I think the result is an argument for RTBC.