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.So, colorbox is used for the filter on admin/content/media and the preview function.
Without colorbox the preview throws an error and the filter page is distorted.
The easiest solution is making colorbox required for enabling MBP.
I like the preview function (should also be present in some way in MBP on a mediafield)
The filter UI is not user friendly this way. (Mentioned here http://drupal.org/node/1038062#comment-4954810 by rootatwc)
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 1355108_add-colorbox-as-dependency_2.patch | 286 bytes | ParisLiakos |
Comments
Comment #1
ParisLiakos CreditAttribution: ParisLiakos commentedAgreed on that, colorbox should be added as a dependency indeed.will try to get a patch
Comment #2
ParisLiakos CreditAttribution: ParisLiakos commentedComment #3
dmsmidtYes, easy enough, but works.
But made me aware that it doesn't work on a version patched with: media_browser_plus-1038062-search-option-2.patch see: http://drupal.org/node/1038062#comment-5300366
Comment #4
das-peter CreditAttribution: das-peter commentedHmm, I'm not sure how to proceed here because of: http://drupal.org/node/1038062#comment-5300366
However, for me a dependency seems reasonable.
But I'd say we should have a feedback from Sebastian.
Comment #5
Sebastian.Buesing CreditAttribution: Sebastian.Buesing commentedI have no problem with making colorbox required! :-)
Comment #7
das-peter CreditAttribution: das-peter commentedPushed: http://drupalcode.org/project/media_browser_plus.git/commit/3d7b014e3ffc...