In MediaLibrary the 'save and insert' is nested in code when it doesn't have to be

Problem/Motivation

In #3034242: Hide "Save and insert" and "Additional selected media" from users by default this was added/changed

+++ b/core/modules/media_library/src/Form/AddFormBase.php
@@ -461,26 +461,32 @@ protected function buildSelectedItemElement(MediaInterface $media, array $form,
-      'save_insert' => [
-        '#type' => 'submit',
-        '#value' => $this->t('Save and insert'),
-        '#ajax' => [
-          'callback' => '::updateWidget',
-          'wrapper' => 'media-library-add-form-wrapper',
-        ],
-      ],
     ];
+    if ($this->isAdvancedUi()) {
+      $actions['save_select']['#value'] = $this->t('Save and select');
+      $actions['save_insert'] = [
+        'save_insert' => [
+          '#type' => 'submit',
+          '#value' => $this->t('Save and insert'),
+          '#ajax' => [
+            'callback' => '::updateWidget',
+            'wrapper' => 'media-library-add-form-wrapper',
+          ],
+        ],
+      ];

As you can see, 'save_insert' is nested in another 'save_insert' wrapper, this seems excessive and makes it weird DX when altering this form.

Proposed resolution

Not sure if we can still remove this, since it would break BC, and no idea how we would deprecate this.

Remaining tasks

Determine if we can/want to fix this

User interface changes

none

API changes

none

Data model changes

the form will have a different structure, which might break code trying to alter this form.

Release notes snippet