This is a simple patch for supporting the resp_img module. Please review...

Comments

attiks’s picture

Status: Needs review » Needs work
+++ b/theme/flexslider.theme.incundefined
@@ -39,6 +39,16 @@ function template_preprocess_flexslider_container(&$vars) {
+    if (module_exists('resp_img')){   ¶

trailing white space

+++ b/theme/flexslider.theme.incundefined
@@ -39,6 +39,16 @@ function template_preprocess_flexslider_container(&$vars) {
+      if (isset($bestfit_suffix) && !empty($bestfit_suffix) && $bestfit_suffix != $default_suffix) { ¶

trailing white space

muschpusch’s picture

Status: Needs work » Needs review
StatusFileSize
new1.05 KB

fixed

attiks’s picture

Status: Needs review » Needs work

something wrong with the patch:
git apply -v flexslider_resp_img_0.patch
fatal: corrupt patch at line 28

muschpusch’s picture

Status: Needs work » Needs review
StatusFileSize
new1.09 KB

ok lazy editing the patch file wasn't good enough. One more rerolled....

attiks’s picture

Status: Needs review » Needs work

I will try to test this later today, but ...

+++ b/theme/flexslider.theme.incundefined
@@ -39,6 +39,17 @@ function template_preprocess_flexslider_container(&$vars) {
+    // resp image support     ¶

white space?

askibinski’s picture

Is the patch really necessary for resp_img support?

I have flexslider and resp_image configured without the patch and it seems to work just fine.
If I'm wrong, could you explain what the benefit would be?

attiks’s picture

@askibinski Did you applied #1512636: Add caption support to Views style plugin, because then it probably will just work without this patch

askibinski’s picture

@attiks ah yes, I did apply that patch, thanks.

attiks’s picture

Status: Needs work » Closed (duplicate)

Marking this as a duplicate in favor of #1512636: Add caption support to Views style plugin