springers54 / reiki

0 stars 0 forks source link

Bildgrößen auf mobilen Geräten stimmen nicht #225

Closed springers54 closed 5 years ago

springers54 commented 5 years ago

grafik

Die Dinge sind leider nicht bearbeitet und auch oben nicht abgehakt.

Im Moment ist es genau falsch rum umgesetzt. Die Bilder auf der Detailseite sind klein, die auf der Übersichtsseite groß.

Bild der Übersichtseite ist aktuell 50% der Breite

grafik

Bild der Detailseite ist aktuell 30% der Seite

grafik

Tabelle mit Bilder mobil zerschossen

grafik

Originally posted by @springers54 in https://github.com/springers54/reiki/issues/185#issuecomment-483128625

springers54 commented 5 years ago

Ich habe hier diesen Teil aus Issue #185 noch einmal rausgezogen. Du hattest zwar den letzten Fehler mit den Tabellen behoben, aber nicht die Bildgröße von Übersichtsseite und Detailseite.

joergsteinhauer commented 5 years ago

Zu "Kleinere Bilder in einen Rahmen setzen, der der Bildbreite entspricht" s. meinen Kommentar hier https://github.com/springers54/reiki/issues/185#issuecomment-486420319

springers54 commented 5 years ago

Funktioniert leider auf dem Handy nicht. Siehe Übersichtseite Heilen, wo ein Buch und ein normaler Beitrag auf einer Seite sind. Man sieht, dass die Texte nicht floaten, also das Buchbild viel zu breit ist. grafik

joergsteinhauer commented 5 years ago

Korrekt, war mein Fehler. Wurde gefixt.

Übrigens, die Aussagen "Man sieht, dass die Texte nicht floaten" ist nicht korrekt. Die Texte in deine, Screenshot floaten (d.h. übrigens "fließen um etwas um"), nur die Bildgröße ist anders.

springers54 commented 5 years ago

Leider ist die Funktionalität nicht konsistent. Auf dieser Seite funktioniert es, auf anderen wieder nicht. Leider dadurch noch schwieriger, dass es mal auf dem Handy nicht stimmt und mal auf dem PC. So befürchte ich, dass die Änderungen für dich wieder problematisch werden und andere Dinge wieder zerschießen, die im Moment passen.

springers54 commented 5 years ago

Folgender Fehler ist aufgefallen:

joergsteinhauer commented 5 years ago

Ja, ist ein Sonderfall, der noch nicht bedacht wurde (konnte ich auch so nicht erahnen). Sollte nun gehen, sobald diese Bilder die Klasse left bekommen.

Das Ausschlaggebende ist dabei, dass die Bilder als als Beitragsbilder ausgegeben werden. Was an sich ja gut ist.

springers54 commented 5 years ago

Du meinst nicht als Beitragsbild, denn genau da funktioniert es nicht. So ist ja die klassische Einbindung bisher bei den Meinungen gewesen. Wir müssen die Bilder direkt als Link im Beitrag vorne einfügen und dann die class=randb oder class=left verwenden, da wir die Bilder auf 150 gebracht haben, ist es egal. Hätten wir mal kleinere, wäre randb besser. Das war bei der Meinung von Peter gemacht und da funktioniert es.

joergsteinhauer commented 5 years ago

Jetzt bin ich baff.... ich könnte schwören, dass ich bei dem "Peter" Beitrag gesehen habe (zumindest im Quellcode), dass das Bild in der Beitragsbild-Box ausgegeben wurde... komisch. Na egal, scheint ja zu funktionieren.