search

ilifau / ili-fau-templates

Landing Page Template for FAU.de
1 stars 1 forks source link

Was bedeutet dies? #18

Closed xwolfde closed 6 years ago

xwolfde commented 6 years ago

"Themenboxen müssen mit einer gültigen Post-ID verknüpft sein."

Wieso? Und was bedeutet dies?

xwolfde commented 6 years ago

Ich hab jetzt mal zwei Themenboxen erstellt, die auf eine Post-ID verweisen.

Jetzt erhalte ich dennoch:

"Warning: implode(): Invalid arguments passed in /var/www/vhosts/wordpress/websource/wp-content/plugins/ili-fau-templates/ILI/FAUTemplates/templates/template-parts/template-topic-boxes.php on line 7"#

Aber wieso überhaupt eine Post-Id?
Die Landing-Oages sind Pages und keine Pages. Die verweisenden Boxen haben irgendeinen Inhalt. Welche Boxen verwendet werden, wird doch in der Metabox bei der Seite angegeben. Wozu soll diese Id gut sein?

SebastianHonert commented 6 years ago

"Post-ID" habe ich ersetzt durch "Verknüpfter Inhalt (ID)". Die Box muss auf einen vorhandenen Inhalt verweisen, sonst wird die ID nicht gespeichert. Die anderen Warnungen sehe ich mir an, das sind alles Kleinigkeiten.

xwolfde commented 6 years ago

Auf was für ein Inhalt denn? Ich versteh es nicht... Und wenn ich es nicht verstehe, dann wird die Webredaktion erst recht planlos sein. Kannst du mal ein Screenshot von einer Konfig bei dir machen?

SebastianHonert commented 6 years ago

Eine Themenbox verlinkt auf einen existierenden Inhalt auf fau.de, der eine eindeutige ID hat.

Es wäre auch möglich, Links als Eingabewert zu akzeptieren, aber die ändern sich ja gelegentlich.

Eine komplexere Lösung wäre die Integration z.B. eines Autocomplete-Feldes, mit dem sich alle Beiträge/Seiten durchsuchen lassen und beim Auswählen automatisch die ID übernommen wird.

xwolfde commented 6 years ago

Ich hab aber die ID eines Posts genommen und es hat nichts bewirkt... Die Box wird nicht angezeigt

SebastianHonert commented 6 years ago

Ich nehme an, Du willst die Box über das Seiten-Template anzeigen und nicht per Shortcode. Hast Du die Box der Seite hinzugefügt?

bildschirmfoto 2018-10-05 um 17 11 57
xwolfde commented 6 years ago

Aaaaahhh... Jetzt hab ich es hinbekommen/verstanden. DIe anzuzeigenen Boxen müssen rechts stehen, nichts links , wo man auswählen soll...

xwolfde commented 6 years ago

Eine Sache ist mir gerade noch aufgefallen: Wenn in dem Content der Themenbox ein Shortcode ist, wird es nicht ausgeführt.

Außerdem ist die Länge des Textes in der Box stark beschränkt. D.h. es wäre momentan nur praktikabel die Box für Bilder einzusetzen, die als Artikelbild definiert wurden. Ich glaub so war es auch definiert. Nur wenn wir das so wollen, dann wäre es besser, bei der Definition des Posttypes der Themenboxen auch den editor für den Content wegzulassen.

SebastianHonert commented 6 years ago

Es wäre nur eine kleine Änderung im Quelltext, um Shortcodes in den Themenboxen zu ermöglichen. Ich denke aber, dass Shortcodes im Anreißertext nicht wirklich etwas verloren haben.

Bzgl. der Länge des Anreißers: Ich habe dem Shortcode einen Parameter "length" hinzugefügt, mit dem sich die Länge des Textes individuell bestimmen lässt.

Außerdem habe ich eine Plugin-Einstellung für die Default-Länge des Anreißers ergänzt (derzeit mind. 10 Zeichen). Dafür habe ich die Einstellung für das Rollenmodell deaktivert, das hatte sowieso noch keine Funktion. Wird das überhaupt benötigt?

SebastianHonert commented 6 years ago

Edit: Parameter geändert in "text_length".

xwolfde commented 6 years ago

Thxs. Ich denke, im ersten Beta können wir darauf verzichten. Feinschliff können wir später immer noch machen. Dann würde ich solche Parameter irgendwo über den Customizer steuerbar machen. Aber erstmal sollte das bisherige optisch passen. Ich vermute, wir werden eh noch einiges an Feedback bekommen, so daß wir uns jetzt nicht zu sehr reinstressen sollten.