[regression] CSS class is not added to MediaLibrary dialog

I've tested it on D11.0.1 using the mentioned steps.
verified that the class "media-library-widget-modal" is attached.

Sharing SS for reference:

I tested it on new D11 instance.

@rhovland have you retested the issue?

I had found another issue talking about this. I had noticed on my Drupal 11 site that the media library modal was narrower than in Drupal 10. I then found the missing class.

https://www.drupal.org/project/drupal/issues/3473586

Changing the line in ckeditor5.ckeditor5.yml certainly fixes the problem.

This is a patch applying the change suggested in the original ticket. It restores the line

dialogClass: media-library-widget-modal

to ckeditor5.ckeditor5.yml.

Thanks for the patch, it works for me on Drupal 11.1.1.

Also works for me on Drupal 11.1.1. Thanks!

Yes, MR changes are working correctly. Moving to review..

Will need test coverage as a bug.

Patch is working for me as well. but not sure why class - ui-dialog-narrow is added here when used in ckeditor
while using media field out of ckeditor does not have this class ? do we also want to remove this class ? if no then may i know what is the use of this ?

We don't have visual regression testing in core so I'm not sure how we'd add test coverage for this, tagging for manual testing though.

We can't add a test to verify the existence of a class?

Seems to be consistently having a test failure though.

I can confirm the issue as I stumbled on this after upgrading to D11.
It's very easy to reproduce with demo_umami profile and opening the add media modal from ckeditor.
I just did a git bisect on the 11.x branch and
66c3914d39bf24d828ec8516389d7cadad52294e is the first bad commit and indeed comes from #3296098: Removal :tabbable usage in dialog.js

Created new MR, the actual fix is in OpenDialogCommand::__construct
- #3296098: Removal :tabbable usage in dialog.js was committed to 10.3.x and 11.x but their commit differs, mainly in OpenDialogCommand::__construct where the issue lies in 11.x, because if $dialog_options['dialogClass'] is set (it is via ckeditor5.js), then $dialog_options['classes']['ui-dialog'] is not taken into account.
- this MR contains improvements in order to ensure both $dialog_options['classes']['ui-dialog'] and $dialog_options['dialogClass'] are taken into account/preserved.
- 10.3.x is not affected by this bug per-se because it does preserve $dialog_options['classes']['ui-dialog'], however it could (should IMO) be ported as well, for consistency.
- I took the opportunity to align the way this is handled in OpenDialogCommand and OpenOffCanvasDialogCommand
- Added some regression tests for those + test to ensure the media-library-widget-modal class is present on the ckeditor media library modal.

Current MR snapshot, for composer.

IS appears to need some love

Thanks @herved for grabbing the latest MR snapshot. Confirmed that this applies correctly in Core 11.1.8

Noting that this fixed and applied cleanly to 11.2.3. Thank you very much

Left some small comments

Cleaning up tags

Test-only feature was ran here https://git.drupalcode.org/issue/drupal-3474018/-/jobs/5763875 so removing that tag.
Summary is updated so removing that tag as well.

The Needs Review Queue Bot tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.

MR rebased

Before

Without the MR this is what I see

After applying

So removing the Needs manual tag.

Most of the feedback appears to be addressed, left 1 thread open. But the issue appears to be fixed.

Attaching a patch that works with 11.2.5

Attaching current MR !12597 diff as static patch usable with 11.3.x.

Hi everyone.
This issue is RTBC since two months. Four Drupal-Versions 11.2.5-11.2.8 has since been released, without the MR being merged. Our customers still see the same small modal as smustgrave described in #40

May I ask what keeps this Issue from being resolved, and wether there is anything I can help?

I've used the patch from #42 on 11.2.7/8 and it works.

@mullzk if you can, use one of the patches with composer install, reach out if you need help.

Committed 462ac97 and pushed to 11.x. Thanks!
Committed 935757d and pushed to 11.3.x. Thanks!

Just upgraded one project to 11.2.9 last week from 10.3.x, and am seeing this. Was there some type of regression, or will this be resolved in 11.3.0?

See comment 48, this was fixed in 11.3