Open springers54 opened 5 years ago
springers54
commented
5 years ago Deine jetzige Variante hat folgenden Effekt:
Im schlimmsten Fall ist jedes Bild unterschiedlich breit, das sieht für eine Übersichtsseite einfach unschön aus.
Ein Problem ist mir aufgefallen, die Buchseiten sind kleiner als 150 px. Hierfür gäbe es 2 Möglichkeiten: Hochskalieren oder das Bild in einen Rahmen setzen, der 150 px breit ist. Letzteres hätte den Vorteil, dass man nicht auch noch mehr Höhe braucht.
joergsteinhauer
commented
5 years ago Habe es gerade gezogen, war ein Missverständnis.
Bei den Buch-Bildern könnte man es so machen.
Ich schlage 2 Varianten vor:
würde komplett automatisch funktionieren, solange das Bild das width Attribut mit 128 hat - ich schaue dann per CSS, welchen Wert dieses Attribut hat (=128) und addiere dann fehlende Abstände und Rahmen, sodass man auf 150px kommt.
joergsteinhauer
commented
5 years ago Mir ist aufgefallen, dass ihr Custom CSS eingefügt habt:
Hier wird margin gesetzt, aber mit komischen Werten. Und vor allem dann nur bei Bildern, die direkt in einem Absatz liegen. Somit kommt es zu Unterschieden, wenn ein bild in einem Link liegt - wie bei Büchern - hier ist der Abstand anders.
Zudem haben durch das custom CSS alle Bilder innerhalb der Absätze einen float. Das kann ganz schnell nach hinten losgehen. Für floats gibts ja die Klassen left etc.
Ich empfehle, dieses Custom CSS zu entfernen.
springers54
commented
5 years ago Alles Custum css, was du bearbeitet hast oder wo die Probleme gelöst sind, kann raus. Das ist ja eher immer eine schnelle Hilfe, ehe du etwas umsetzt.
joergsteinhauer
commented
5 years ago Ok.
Rahmen ist gesetzt und die Gesamtbreite des Bildes ist dann 150px, wenn die Bilder kleiner als 150px sind. Das ging leider nur mit JS. Bitte überprüfen.
springers54
commented
5 years ago hast du selbst mal geguckt? Wäre doch der erste Schritt.
https://zen-reiki.de/themen/bereich/heilen/
Also bei uns kommt jetzt kein Rahmen und die Bilder sind nicht 150 und unterschiedlich breit, die Texte springen.
joergsteinhauer
commented
5 years ago Jetzt sollte es passen
springers54
commented
5 years ago Siehe #185 . leider nur auf dem Desktop im Moment
springers54
commented
5 years ago Im Moment stimmt es auch auf dem Desktop mal wieder nicht mehr, siehe letzter Kommentar in #185
Desktop:
Auf der Übersichtsseite wollten wir kleinere Bilder habe, am besten, auch die kleinere Version laden, wobei du meintest, das geht nicht und auch hier die großen Bilder lädst und diese nur runterscalierst.
Die Breite der Bilder auf der Übersichtsseite ist hier 150 px.
Es war vorher auch so:
Jetzt ist es wieder anders, nachdem ich meinen Cache gelöscht habe:
Damit entsprechen die Bilder der Übersichtsseite der der Detailseite und das macht keinen Sinn.