Plugin cssMarquee [Testversion]

Ein CMSimple Support Forum für deutsch sprechende Nutzer und Entwickler
lck
Posts: 2967
Joined: Wed Mar 23, 2011 11:43 am
Contact:

Re: Plugin cssMarquee [Testversion]

Post by lck » Sun Jan 15, 2023 5:41 pm

cmb wrote:
Sun Jan 15, 2023 3:08 pm
Das sieht unnötig kompliziert aus, da die Zuweisung von $message unabhängig von der Ausrichtung (vertikal oder horizontal) immer gleich ist. Sollte auch so gehen:
Für dich ja. ;) , für mich war's ok. Getestet, funktioniert.
cmb wrote:
Sun Jan 15, 2023 1:38 pm
Alternativ könnte man auch die Möglichkeit bieten, dass User den gewünschten Inhalt auf einer News-Seite eingeben, und diesen anzeigen. Etwa:
[...]
Das ist natürlich noch besser. Einfacher für den Benutzer als in der Sprachdatei des Plugins das "händisch" zu machen. Obwohl, ich hab da auch Vorlagen vorgegeben und die könnte man auch noch erweitern.
Ich werde das mit den Newsboxen mal testen. Die Idee finde ich gut.
„Bevor du den Pfeil der Wahrheit abschießt, tauche die Spitze in Honig!“   👉 Ludwig's XH-Templates for MultiPage & OnePage

olape
Posts: 2745
Joined: Fri Mar 13, 2015 8:47 am
Contact:

Re: Plugin cssMarquee [Testversion]

Post by olape » Sun Jan 15, 2023 6:11 pm

Müsste man bei den Newsboxen nicht aber html-Tags wie hx, p, div, br, hr rausfiltern?
Gruß Olaf, Plugins for CMSimple_XH

Ich habe schon lange den Verdacht, dass so viele so eifrig auf Gender, Trans und Queer machen:
Weil sie für das Fachliche ganz einfach zu doof sind.

lck
Posts: 2967
Joined: Wed Mar 23, 2011 11:43 am
Contact:

Re: Plugin cssMarquee [Testversion]

Post by lck » Sun Jan 15, 2023 7:19 pm

olape wrote:
Sun Jan 15, 2023 6:11 pm
Müsste man bei den Newsboxen nicht aber html-Tags wie hx, p, div, br, hr rausfiltern?
Warum meinst du müsste/sollte man das?
„Bevor du den Pfeil der Wahrheit abschießt, tauche die Spitze in Honig!“   👉 Ludwig's XH-Templates for MultiPage & OnePage

olape
Posts: 2745
Joined: Fri Mar 13, 2015 8:47 am
Contact:

Re: Plugin cssMarquee [Testversion]

Post by olape » Mon Jan 16, 2023 6:32 am

lck wrote:
Sun Jan 15, 2023 7:19 pm
olape wrote:
Sun Jan 15, 2023 6:11 pm
Müsste man bei den Newsboxen nicht aber html-Tags wie hx, p, div, br, hr rausfiltern?
Warum meinst du müsste/sollte man das?
Ich vermute, dass diese Tags für das Layout im Marquee störend sein könnten.
Gruß Olaf, Plugins for CMSimple_XH

Ich habe schon lange den Verdacht, dass so viele so eifrig auf Gender, Trans und Queer machen:
Weil sie für das Fachliche ganz einfach zu doof sind.

lck
Posts: 2967
Joined: Wed Mar 23, 2011 11:43 am
Contact:

Re: Plugin cssMarquee [Testversion]

Post by lck » Mon Jan 16, 2023 11:48 am

olape wrote:
Mon Jan 16, 2023 6:32 am
Ich vermute, dass diese Tags für das Layout im Marquee störend sein könnten.
Ja, da hast du schon recht, teilweise. Störend in dem Sinn nicht. Die Block-Elemente (der Content) werden mittels Flexbox per "row" bzw. vertical per "column" ausgegeben. Aber, die unterschiedlichen margins und padding's die in der Template CSS festgelegt sind, sind eins, weil der Text nicht auf gleicher Höhe ist. Zumindest im horizontalen Marquee und hier speziell die Überschriften. Vertikal sehe ich momentan keine Probleme.

:? Abfangen könnte ich das per CSS schon. Margins und paddings eben auf 0 setzen.
„Bevor du den Pfeil der Wahrheit abschießt, tauche die Spitze in Honig!“   👉 Ludwig's XH-Templates for MultiPage & OnePage

olape
Posts: 2745
Joined: Fri Mar 13, 2015 8:47 am
Contact:

Re: Plugin cssMarquee [Testversion]

Post by olape » Mon Jan 16, 2023 1:23 pm

lck wrote:
Mon Jan 16, 2023 11:48 am
:? Abfangen könnte ich das per CSS schon. Margins und paddings eben auf 0 setzen.
Ich denke es könnte einfach so funktionieren

Code: Select all

$array_search = array('<h1>',
                      '</h1>',
                      '<h2>',
                      '</h2>',
                      '<h3>',
                      '</h3>',
                      '<h4>',
                      '</h4>',
                      '<h5>',
                      '</h5>',
                      '<h6>',
                      '</h6>',
                      '<hr>',
                      '<hr />',
                      '<br>',
                      '<br />',
                      '<p>',
                      '</p>'
);
$text = str_ireplace($array_search, '', $text);
Gruß Olaf, Plugins for CMSimple_XH

Ich habe schon lange den Verdacht, dass so viele so eifrig auf Gender, Trans und Queer machen:
Weil sie für das Fachliche ganz einfach zu doof sind.

cmb
Posts: 14225
Joined: Tue Jun 21, 2011 11:04 am
Location: Bingen, RLP, DE
Contact:

Re: Plugin cssMarquee [Testversion]

Post by cmb » Mon Jan 16, 2023 1:58 pm

olape wrote:
Mon Jan 16, 2023 1:23 pm
Ich denke es könnte einfach so funktionieren
Oder alternativ strip_tags() mit explizit erlaubten Elementen. Oder halt einfach den Usern sagen, dass bestimmtes Markup nicht wirklich unterstützt wird (testen was geht können sie ja dann selbst).
Christoph M. Becker – Plugins for CMSimple_XH

olape
Posts: 2745
Joined: Fri Mar 13, 2015 8:47 am
Contact:

Re: Plugin cssMarquee [Testversion]

Post by olape » Mon Jan 16, 2023 2:15 pm

cmb wrote:
Mon Jan 16, 2023 1:58 pm
Oder halt einfach den Usern sagen, dass bestimmtes Markup nicht wirklich unterstützt wird (testen was geht können sie ja dann selbst).
Von solchen Gedanken habe ich mich schon lange verabschiedet, es gilt leider fast immer und überall; abfangen was irgendwie geht. :cry:
cmb wrote:
Mon Jan 16, 2023 1:58 pm
Oder alternativ strip_tags() mit explizit erlaubten Elementen.
Ja, das wäre auch ein Weg.

Aber ich glaube, beide Varianten werden so in eingen Situationen noch nicht ganz korrekt funktionieren.
Wenn ich i.e. zwei aufeinanderfolgende Absätze habe:

Code: Select all

<p>absatz1</p>
<p>absatz2</p>
dann würde daraus
"absatz1absatz2" und nicht "absatz1 absatz2"

Man müsste also bei meinem Vorschlag noch die Zeile:

Code: Select all

$text = str_ireplace($array_search, '', $text);
durch:

Code: Select all

$text = str_ireplace($array_search, ' ', $text);
$text = preg_replace('#[\s]{2,}#s', ' ', $text);
ersetzen
Gruß Olaf, Plugins for CMSimple_XH

Ich habe schon lange den Verdacht, dass so viele so eifrig auf Gender, Trans und Queer machen:
Weil sie für das Fachliche ganz einfach zu doof sind.

lck
Posts: 2967
Joined: Wed Mar 23, 2011 11:43 am
Contact:

Re: Plugin cssMarquee [Testversion]

Post by lck » Mon Jan 16, 2023 4:32 pm

Erstmal danke, dass sich so viele hier beteiligen. So soll's sein, das macht ein Forum aus.
olape wrote:
Mon Jan 16, 2023 1:23 pm
lck wrote:
Mon Jan 16, 2023 11:48 am
:? Abfangen könnte ich das per CSS schon. Margins und paddings eben auf 0 setzen.
Ich denke es könnte einfach so funktionieren
[...]
Wo genau müsste ich das in der index.php einfügen und vermutlich muss $text durch $message ersetzt werden. Hab das an alle möglichen Stellen gesetzt aber ohne Wirkung.

Ehrlich gesagt, will/braucht ich/es diese Filterung gar nicht. Dann wäre der Sinn einer Einbindung einer Newsbox und die Vorteile des Editors doch eigentlich umsonst. Es reicht das margin und padding oben und unten zu entfernen um die Elemente auf gleiche Linie zu bringen. Wie bereits erwähnt, in der vertikalen Ausgabe passt es eh. Das hat der User auch selber in der Hand und kann das nach seinem Gusto gestalten.
cmb wrote:
Mon Jan 16, 2023 1:58 pm
Oder halt einfach den Usern sagen, dass bestimmtes Markup nicht wirklich unterstützt wird (testen was geht können sie ja dann selbst).
+1 Und wenn jemand damit nicht klarkommt, so findet er hier bestimmt Hilfe. :)

BTW: Mittlerweile ist das kein Marquee mehr, im eigentlichen Sinne, sondern wird eher eine Art "Content-Slider". 8-)
„Bevor du den Pfeil der Wahrheit abschießt, tauche die Spitze in Honig!“   👉 Ludwig's XH-Templates for MultiPage & OnePage

olape
Posts: 2745
Joined: Fri Mar 13, 2015 8:47 am
Contact:

Re: Plugin cssMarquee [Testversion]

Post by olape » Mon Jan 16, 2023 5:41 pm

lck wrote:
Mon Jan 16, 2023 4:32 pm
Wo genau müsste ich das in der index.php einfügen und vermutlich muss $text durch $message ersetzt werden.
Das kann ich so nicht sagen, da ich deinen aktuellen Entwurf nicht kenne.
Es geht um den Inhalt der Newsbox.
Also am besten gleich nachdem die Box eingelesen ist.
Und ja, wenn diese Variable bei dir „message“ heißt, dann …
Gruß Olaf, Plugins for CMSimple_XH

Ich habe schon lange den Verdacht, dass so viele so eifrig auf Gender, Trans und Queer machen:
Weil sie für das Fachliche ganz einfach zu doof sind.

Post Reply