benjaminkott / bootstrap_package

Bootstrap Package delivers a full configured theme for TYPO3, based on the Bootstrap CSS Framework.
https://www.bootstrap-package.com/
MIT License
338 stars 205 forks source link

Image gallery: breakpoints for 2 or 3 imagecols missing + low image resolution #1498

Closed christophbee closed 3 months ago

christophbee commented 3 months ago

Bug Report

Prerequisites

Description

It's about the content element 'Images' (CType 'image') where you can choose the number of image columns (field 'imagecols'). When choosing 2 or 3 columns the breakpoint is missing in CSS + the image resolution is too low for switching to 2 or 1 columns on mobile devices.

Steps to Reproduce

  1. use a content element of type "Images"
  2. Add 3 images
  3. choose 3 imagecols
  4. check the responsive behaviour in frontend

Expected behavior

At breakpoint 'sm' the --gallery-columns shall change to 1 optional: At breakpoint 'md' the --gallery-columns shall change to 2

Additionally the image resolution is expected to adopt to the new column sizes.

Actual behavior

There is no breakpoint defined in CSS for 2 and 3 image columns, see screenshot.

Screenshots

Bildschirmfoto 2024-07-25 um 13 35 19

3 columns on larger devices: Bildschirmfoto 2024-07-25 um 13 32 19

manually changed to 2 columns at breakpoint 'md' -> low image resolution Bildschirmfoto 2024-07-25 um 13 32 27

manually changed to 1 column at breakpoint 'sm' -> low image resolution Bildschirmfoto 2024-07-25 um 13 32 33

Versions

TYPO3 v12.4.17 bootstrap_package v14.0.7

christophbee commented 3 months ago

UPDATE: I found to adjust the images' size by setting the multiplier in typoscript via

lib.contentElement.settings{
    gallery{
        columns{
            2{
                    multiplier {
                        small = 1
                        extrasmall = 1
                    }
                }
            3{
                multiplier {
                    medium = 1
                    small = 1
                    extrasmall = 1
                }
            }
        }
    } 
}
christophbee commented 3 months ago

For my case the issue is solved but maybe it's a reason to modify image rendering a little bit in future versions.