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.
If the view used in the popup dialog has lots of exposed filters, or lists lots of metadata under each image, then the pager or even some view items can get cut off by the bottom of the iframe.
This is owing to the fact that lots of elements in the iframe (including the iframe) are restricted to 446px high (or some variation on it.) These are hardcoded in e.g. imagebrowser.css and .js, and are therefore difficult to override with your own themeing.
I'll attach a patch in the first comment to this ticket which exposes the magic "446" as a Drupal system variable.
Comment | File | Size | Author |
---|---|---|---|
#5 | 1393642-imagebrowser-paging-inline.patch.1.txt | 316 bytes | jp.stacey |
#3 | 1393642-imagebrowser-dialog-cutoff.patch.3.txt | 4.31 KB | jp.stacey |
#2 | 1393642-imagebrowser-dialog-cutoff.patch.2.txt | 4.3 KB | jp.stacey |
#1 | 1393642-imagebrowser-dialog-cutoff.patch.1.txt | 4.85 KB | jp.stacey |
Comments
Comment #1
jp.stacey CreditAttribution: jp.stacey commentedPlease find attached a patch which exposes "446" as a Drupal system variable
imagebrowser_height
, then pipes it through tobrowser.tpl.php
and theDrupal.settings.imagebrowser.baseheight
setting.Comment #2
jp.stacey CreditAttribution: jp.stacey commentedBah, a hunk in the patches to the JS files failed on one of those trailing/not-trailing newlines. Reattaching.
Comment #3
jp.stacey CreditAttribution: jp.stacey commentedAnd again - Javascript casts numbers to strings during addition, so ensure
Drupal.settings.imagebrowser.baseheight
is numeric.Comment #4
jp.stacey CreditAttribution: jp.stacey commentedHere's a separate, one-line patch to the pager styling, which currently leaves the
display
of all the pager li items to the browser (so typically block display.)Comment #5
jp.stacey CreditAttribution: jp.stacey commentedSorry, separate patch attached now.