Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Problem with responsive images #8807

Closed
mlwebworker opened this issue Oct 26, 2017 · 5 comments
Closed

Problem with responsive images #8807

mlwebworker opened this issue Oct 26, 2017 · 5 comments

Comments

@mlwebworker
Copy link

Es geht um die neue Variante bei Pixeldichte/Skalierungsfaktor - die Breitenbeschreibung (z.B. 300w), die ich sehr begrüße.
Bei einem Test ist mir allerdings zufällig aufgefallen, dass ein für mich nicht nachvollziehbares srcset entsteht, falls die angegebene Breite größer ist als die Breite des Originalbildes.
Beschrieben im Forum https://community.contao.org/de/showthread.php?68549-Responsive-geht-gar-nicht&p=454886&viewfull=1#post454886.
Da wir grundsätzlich nicht ausschließen können, dass Redakteure das auf zu kleine Originalbilder anwenden, sollte dieser Fall m.E. genauso abgefangen werden wie das passiert, wenn ich für ein Bild 600w angebe aber das Originalbild diese Breite gar nicht hergibt.

@fritzmg
Copy link
Contributor

fritzmg commented Oct 26, 2017

Du solltest die Fehlerbeschreibung hier im Ticket posten.

@mlwebworker
Copy link
Author

mlwebworker commented Oct 26, 2017

Du hast ja recht.
Getestet wurde mit zwei Bildern
Bild 1: Originalbreite 600px
Bild 2: Originalbreite 477px

Eintrag im Backend bei Bildgrößen (Variante 1):
Breite: 600
size: 100vw
Pixeldichte/Skalierungsfaktor: 250w, 300w, 400w, 600w
Für das Bild mit einer Originalbreite war das erzeugte srcset wie erwartet.
Für das Bild mit einer Originalbreite von 477px ergab sich im Code ein srcset mit 477w, 377w und 314w.
Sicher korrekt ist, dass keine nach oben skalierten Bilder berechnet werden sollen, aber auf die Werte 377w und 314w kann ich mir spontan keinen Reim machen.

Eintrag im Backend bei Bildgrößen (Variante 2):
Breite: 400
size: 100vw
Pixeldichte/Skalierungsfaktor: 250w, 300w, 400w, 600w
Für beide Bilder ist das erzeugte srcset wie erwartet.
Für das zu kleine Originalbild wird 600w völlig korrekt ignoriert und auch erfreulicher Weise ein Eintrag mit der Originalbreite (477w) erzeugt.

@Toflar
Copy link
Member

Toflar commented Oct 27, 2017

/cc @ausi

@ausi
Copy link
Member

ausi commented Oct 27, 2017

Ich kann das Problem reproduzieren, ist ein Fehler in contao/image.

@leofeyer kannst du das Issue bitte nach contao/image verschieben?

@ghost
Copy link

ghost commented Oct 27, 2017

Das Ticket wurde nach contao/image#39 verschoben.

@ghost ghost closed this as completed Oct 27, 2017
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants