sms-responsive-images

sms-responsive-images copied to clipboard

dmitryd

The extension seem to work ok on v13.

dmitryd

If i use <picture> tag with sources with "srcset" and "sizes", and enable lazy loading using the native loading="lazy" attribute, Chrome does not load the images on page load. Adding...

sir-gawain

I am upgrading from 2.1.3 to 3.0.0. Before the upgrade i could deliver a static image width via the `width` attribute. After the upgrade the `width` attribute is completely ignored...

rov-sf

Modified Tests for additional height parameter

5

Should solve issue #48

naturefund-falk

Is there any reason you don't add a height attribute to fallback image?

3

https://github.com/sitegeist/sms-responsive-images/blob/1a559c160c0e260c17ad2f974365807748c34b7f/Classes/Utility/ResponsiveImagesUtility.php#L221 You could also do ` $fallbackTag->addAttribute('height', $fallbackImage->getProperty('height')); ` like it's done in f:media. Or am I missing something?

ktallafus

Respect CropArea while generating placeholder image

3

Images can be cropped to fit a certain aspect ratio. To display a placeholder image with a correct aspect ratio, the crop area should be used. Currently the original size...

brosua

The extension adds a few TypoScript options to `lib.contentElement` which should be explained in the documentation. How to specify those options per column should also be mentioned: https://techblog.sitegeist.de/responsive-images-with-typo3-8-7/#comment-4

s2b

Add option to specify image quality

4

It should be possible to specify/overwrite the image quality option. Ideally, it should be possible to specify multiple qualities, e. g. to generate lower-quality for high dpi images.

s2b

Add Support for maxHeight attribute

1

Currently the maxHeight attribute is ignored when in srcset mode.

naturefund-falk