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.
With latest jQuery Update and jQuery UI 1.10.2 media browser doesn't change it's height which is stuck to 280 px due to Drupal.media.browser.resizeIframe
which is never called.
The reason of this is API change in jQuery UI 1.10.x: Tabs Widget API has no "show" event anymore according to: http://bugs.jqueryui.com/ticket/7155
Comment | File | Size | Author |
---|---|---|---|
#10 | media_browser-1957812-10.patch | 1.03 KB | sylus |
#9 | media_browser-1957812-09.patch | 1001 bytes | sylus |
#8 | media_browser_-1957812-08.patch | 1001 bytes | sylus |
#1 | media_browser-1957812-1.patch | 1 KB | OnkelTem |
Comments
Comment #1
OnkelTem CreditAttribution: OnkelTem commentedComment #2
avr CreditAttribution: avr commentedRan into this issue - this looks & works well for me.
Might as well also add:
so that the dialog is resized on initial load (sorry - didn't have a good setup to reroll the patch). This also helps if you have an error/message in your form submission.
Comment #3
gagarine CreditAttribution: gagarine commentedDuplicate of #1956634: Media browser display is very narrow with jQuery 1.8
Comment #4
Rob C CreditAttribution: Rob C commentedRe-opening, because this issue already has a patch available. (while it is newer)
Comment #5
Rob C CreditAttribution: Rob C commentedAnd https://drupal.org/node/1956634#comment-7532673 looks true. (after some testing)
Update to the latest jquery_update 2.x-dev (or git) and configure jquery 1.8, then you do not need to patch the media module again. So closing up again. Jquery_update 2.4 would prolly fix this once released.
Comment #6
stefan.kornIt seems to me that it still does not work with the latest dev of jquery update: 7.x-2.3+6-dev.
Is there some place I can find a newer dev version?
Comment #7
OnkelTem CreditAttribution: OnkelTem commentedActually, jQuery Update has nothing to do with API. It had changed and can't magically start working :)
Btw, there is one more issue with the latest Media dev version:
The line marked "problem" should read as:
to work on latest jQuery UI.
Comment #8
sylus CreditAttribution: sylus commentedAttaching updated patch based on discussion above.
Comment #9
sylus CreditAttribution: sylus commentedWhoops removed extra underscore.
Comment #10
sylus CreditAttribution: sylus commentedAdding the create from comment: #2
Everything is resizing and working perfectly now :)
Should be good to go.
Comment #11
OnkelTem CreditAttribution: OnkelTem commentedNote that you just deleted:
- $('#media-browser-tabset').tabs('select', Drupal.media.browser.activeTab);
and added:
+ $('#media-browser-tabset').tabs('option', 'active', Drupal.media.browser.activeTab);
which is not compatible with ancient Drupal 7 jQuery :)
Comment #12
sylus CreditAttribution: sylus commentedOh I didn't even realize that thanks for the catch! How would you recommend solving this?
Comment #13
OnkelTem CreditAttribution: OnkelTem commentedI'm not aware of a common Drupal way for doing this. I would check $.ui.version, maybe something like:
Comment #14
ianthomas_ukThe issue described is the same as https://drupal.org/node/2032255 which has a patch for both versions of jQuery UI.
The issue that the second half of the patch is attempting to fix is a separate incompatibility, so I've raised a new issue for that - https://drupal.org/node/2089913
Comment #15
ianthomas_ukI've uploaded a patch to https://drupal.org/node/2089913
Comment #15.0
ianthomas_ukUpdated issue summary.