Skip to content

Instantly share code, notes, and snippets.

@opi
Created April 26, 2021 11:14
Show Gist options
  • Star 4 You must be signed in to star a gist
  • Fork 1 You must be signed in to fork a gist
  • Save opi/b855eb28cbeb4d01abe125e8c978d978 to your computer and use it in GitHub Desktop.
Save opi/b855eb28cbeb4d01abe125e8c978d978 to your computer and use it in GitHub Desktop.
Drupal: Do not put buttons into modal/dialog button pane

Do not put buttons into modal/dialog button pane

See https://www.drupal.org/project/drupal/issues/3089751 & https://www.drupal.org/forum/support/theme-development/2018-12-08/prevent-form-actions-from-appearing-in-a-modal-dialog

In mytheme.libraries.yml

dialog.ajax:
  js:
    path/to/js/dialog.ajax.js: { minified: true }

In mytheme.info.yml

libraries-extend:
  core/drupal.dialog.ajax:
    - mytheme/dialog.ajax

Create a file called path/to/js/dialog.ajax.js in your theme, with the following content :

(function (Drupal) {
  // Override core/dialog.ajax prepareDialogButtons behaviors
  Drupal.behaviors.dialog.prepareDialogButtons = function prepareDialogButtons($dialog) {
    // Do nothing = do not put buttons into dialog footer
    return [];
  }
})(Drupal);
@jfremer
Copy link

jfremer commented Jul 13, 2021

Thank you for posting this. I was pulling what remains of my hair out. 🍻

@nimoatwoodway
Copy link

🍻 Thanks!

@mdance
Copy link

mdance commented Nov 10, 2021

I've come up with a better solution, you can selectively add a class to your $form['actions']['#attributes']['class'][] = 'disable-dialog-buttons'; to disable this functionality on a per form basis...

(function ($, Drupal) {
  Drupal.behaviors.dialog.prepareDialogButtons = function prepareDialogButtons($dialog) {
    // Check for a class in the form actions container to disable this
    // functionality
    if ($dialog.find('.form-actions.disable-dialog-buttons').length) {
      return [];
    }

    var buttons = [];
    var $buttons = $dialog.find('.form-actions input[type=submit], .form-actions a.button');
    $buttons.each(function () {
      var $originalButton = $(this).css({
        display: 'none'
      });
      buttons.push({
        text: $originalButton.html() || $originalButton.attr('value'),
        class: $originalButton.attr('class'),
        click: function click(e) {
          if ($originalButton.is('a')) {
            $originalButton[0].click();
          } else {
            $originalButton.trigger('mousedown').trigger('mouseup').trigger('click');
            e.preventDefault();
          }
        }
      });
    });
    return buttons;
  }
})(jQuery, Drupal);

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment