The configured dialog height is overrided to be auto

Created on 12 October 2024, 3 months ago

Problem/Motivation

Found in https://www.drupal.org/project/experience_builder/issues/3471978 📌 Media Library dialog styling Active

In dialog.position.js, in the resetSize() function we have the following logic:

          if (
            option === 'height' &&
            event.data.$element.parent().outerHeight() < adjustedValue
          ) {
            adjustedValue = 'auto';
          }

which basically makes the height option function as maxHeight. I think this behaviour is incorrect and also makes it hard to have dialog with iframes in it (because they use the height they have available and not the height it requires)

I think we just should remove the lines above, and recommend someone to use maxHeight and height = 'auto' instead.

Happy to put this behind a BC option if that makes sense.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

🐛 Bug report
Status

Active

Version

11.0 🔥

Component

javascript

Created by

🇸🇪Sweden johnwebdev

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024