Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Currently, only elements with the #type "submit" are displayed in the "Operations" column of theme_media_widget_multiple. Other buttons included with the media_generic widget like Edit, Browse, Preview, and Attach are "link" elements, but should also be displayed in the "Operations" column. This patch makes sure that all link and submit elements are placed in the "Operations" column in theme_media_widget_multiple.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | media-operation-column-links-2476205-1.patch | 672 bytes | samuel.mortenson |
Comments
Comment #1
samuel.mortensonComment #2
Devin Carlson CreditAttribution: Devin Carlson commentedThanks for the patch!
Tested and committed to Media 7.x-2.x.
Comment #4
jurgenhaasThis patch unfortunately breaks other functionality, e.g. the project epsacrop. They are looking for the class "media-widget" and within that context they then try to find crop buttons. But with this change the crop button moves out to a different context and then their functionality doesn't work anymore.
Comment #5
jurgenhaasWhat is going to happen about this issue? We got hit by this problem again because of the media 2.x-beta1 upgrade.
Comment #6
samuel.mortensonI would recommend opening an issue with epsacrop, it sounds like this could be fixed by a few changes to their JS selectors. As far as I know no other projects have been affected by this change.
Comment #7
jurgenhaasFair point and it appears there is already a patch for it, see #2506721: Manage image crops button not working with unlimited media fields
Comment #8
samuel.mortensonI'm glad the solution was already out there. :-) I'll mark this issue as Fixed, if anyone else has problems feel free to re-open it.