Willst du jetzt bei mir Zuhause vorbeikommen?Aber ohne die Site mal zu sehen …
Die Seite ist noch nicht online....
Willst du jetzt bei mir Zuhause vorbeikommen?Aber ohne die Site mal zu sehen …
Na dann müssen wir wohl auf den Mann mit der Glaskugel warten.
Ich vermute du hast nicht den .iframe_container sondern .iframe_container ifram geändert.
Code: Select all
.forum_comment .iframe_container {
position: relative;
overflow: hidden;
width: 75%;
padding-top: 75%; /* 16:9 Aspect Ratio (divide 9 by 16 = 0.5625) */
}
Ja, klar! Das ist doch die Idee bei Open-Source – man kann nach Bedarf anpassen, und anderen zur Verfügung stellen.lck wrote: ↑Tue Jul 06, 2021 7:41 pmTeste mal diese modifizierte Version. Ich hoffe Christoph ist das recht .
Klappt! Begeisterung!lck wrote: ↑Wed Jul 07, 2021 5:55 pmIch vermute du hast nicht den .iframe_container sondern .iframe_container ifram geändert.
Also so ändern:Das ist dann aber nicht mehr responsiv (16:9). Wenn dann müsste man bei 75% Breite, den Wert für padding-top auf 42.19% (=75 x 0.5625) setzen.Code: Select all
.forum_comment .iframe_container { position: relative; overflow: hidden; width: 75%; padding-top: 75%; /* 16:9 Aspect Ratio (divide 9 by 16 = 0.5625) */ }
Auch die Änderung klappt!cmb wrote: ↑Wed Jul 07, 2021 11:26 pmBzgl. der Link-Problematik: da hatte ich alle in die Irre geführt, sorry! Einen hoffentlich korrekten Fix habe ich gerade eingecheckt.
Ja, klar! Das ist doch die Idee bei Open-Source – man kann nach Bedarf anpassen, und anderen zur Verfügung stellen.lck wrote: ↑Tue Jul 06, 2021 7:41 pmTeste mal diese modifizierte Version. Ich hoffe Christoph ist das recht .
Und wenn du nichts dagegen hast, würde ich deinen Iframe-Code gerne übernehmen.
Ja klar, natürlich. Danke!cmb wrote: ↑Wed Jul 07, 2021 11:26 pmUnd wenn du nichts dagegen hast, würde ich deinen Iframe-Code gerne übernehmen.
Code: Select all
return '<div class="iframe_container"><iframe src="' . $url . '" title="' . basename($url) . '"></iframe></div>';
Code: Select all
return '<div class="iframe_container ' . basename($url) . '"><iframe src="' . $url . '" title="' . basename($url) . '"></iframe></div>';
Super!lck wrote: ↑Thu Jul 08, 2021 12:26 pmJa klar, natürlich. Danke!cmb wrote: ↑Wed Jul 07, 2021 11:26 pmUnd wenn du nichts dagegen hast, würde ich deinen Iframe-Code gerne übernehmen.
Was die Klasse angeht, kannst du einfach ein Präfix verwenden; aber das löst nicht das Problem, dass CSS-Klassennamen wohl diverse Zeichen, die in einem URL-Pfad vorkommen können, nicht unterstützen. Evtl. vielleicht md5($url) (statt basename($url)), und dann ein Präfix? Vielleicht tut's schon ein crc32.lck wrote: ↑Thu Jul 08, 2021 12:26 pmNoch ein Nachtrag. In der BBCode.php habe ich die Zeile 142noch erweitert, um eine individuelle Klasse. So wäre es möglich die iFrames in speziellen Fällen auch anders zu gestalten.Code: Select all
return '<div class="iframe_container"><iframe src="' . $url . '" title="' . basename($url) . '"></iframe></div>';
Problem: bei einem Video mit einer Ziffer am Anfang, Beispiel 7LXpnNKNxJI. Klassennamen dürfen nicht mit einer Ziffer beginnen.Code: Select all
return '<div class="iframe_container ' . basename($url) . '"><iframe src="' . $url . '" title="' . basename($url) . '"></iframe></div>';
Auch den title sollte man nochmal überdenken, wäre dann auch 7LXpnNKNxJI. Gut das stört eigentlich wenig und ein title im iFrame muss sein.
Ja, das ginge. Wird halt eine langer Klassenname.cmb wrote: ↑Thu Jul 08, 2021 12:55 pmWas die Klasse angeht, kannst du einfach ein Präfix verwenden; aber das löst nicht das Problem, dass CSS-Klassennamen wohl diverse Zeichen, die in einem URL-Pfad vorkommen können, nicht unterstützen. Evtl. vielleicht md5($url) (statt basename($url)), und dann ein Präfix?
Code: Select all
return '<div class="iframe_container ic_' . md5($url) . '"><iframe src="' . $url . '" title="' . basename($url) . '"></iframe></div>';
Da mutest du mir zuviel zu .
Ich würde sagen, warten wir mal ab was Ulrich zu der dem Ganzem sagt. Ich glaube, er ist bereits glücklich so wie es ist.