\Drupal\media_library\MediaLibraryUiBuilder::dialogOptions dialogClass should be js- prefixed since the media modal JavaScript depends on this

Created on 28 April 2020, over 5 years ago
Updated 18 August 2025, about 2 months ago

Follow up to 📌 MediaLibraryWidget should use the media_library.ui_builder service when fetching modal dialog options Needs work

In the parent issue a comment was added that the dialogClass set in \Drupal\media_library\MediaLibraryUiBuilder::dialogOptions is required for the proper functioning of the media modal. But to make this clearer, we should prefix the class with js-

Alternatively this should use a data-attribute and not a class.

📌 Task
Status

Active

Version

11.0 🔥

Component

media system

Created by

🇳🇱Netherlands Lendude Amsterdam

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

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024