The current trigger field does not allow for colorbox to auto resize the window based on the size of the content within it. If you clear our the default values in the Views UI for the width and height of the window, you do not get the expected results. The render method for the field has the window size hard coded. This patch simply removes the assumption that we want a specific size for our colorbox pop-up.

CommentFileSizeAuthor
#1 views-colorbox-size-1296528-1.patch1.2 KBtom friedhof
views_colorbox_size.patch1.2 KBtom friedhof
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tom friedhof’s picture

tom friedhof CreditAttribution: tom friedhof commented
FileSize
views-colorbox-size-1296528-1.patch1.2 KB

Uploading the same patch as above with the issue# in patch file name so I can find this issue easily.

frjo’s picture

frjo CreditAttribution: frjo commented
Status: Active » Fixed

Committed to 7-dev.

Thanks for contributing this fix!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.