cl_image_tag adds non-dynamic height/width for :crop => :fit

[aaronchi]

When using cl_image_tag with :crop => :fit, it passes the non-dynamic height/width variables to the image tag which makes the image size incorrectly.

For example:

cl_image_tag(bonus.image.filename, :width => 250, :height => 250, :crop => :fit)

This always produces an image tag with height=250 and width=250 regardless of how the image is cropped. cl_image_tag should not pass through the height and width if crop is set to fit or should use a different variable to pass the cloudinary vs the image options.

[nadavs]

Thank you for reporting this issue.
We should indeed fix cl_image_tag not to add the width & height attributes when 'fit' or 'limit' crop modes are used.
In the meantime, there's an easy workaround. Simply use :html_width and :html_height. For example:

cl_image_tag("sample.jpg", :width => 250, :height => 250, :crop => :fit, :html_width => nil, :html_height => nil)

[nadavs]

This issue was fixed in v1.0.34. Width & Height are not passed to the image tag in 'fit' and 'limit' crop modes.