ilifau / ili-fau-templates

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

Landingpage: besseres Defaultverhalten #23

Closed xwolfde closed 5 years ago

xwolfde commented 6 years ago

Wenn man eine neue Landingpage anlegt ist noch kein erstes Bild oben angegeben. Trotzdem wird beim Aufruf schon was gezeigt. Allerdings "nichts":

image

Vorschläge für besseres Defaultverhalten und zur Kompatibilität:

Wenn noch kein erster Slider ausgewählt ist, dann schaue ob nicht stattdessen ein Artikelbild für die Seite definiert ist. Wenn ja, nehme dies. Wenn weder Slides vorhanden sind, noch ein Artikelbild, dann schaue ob es ein globales Ersatzbild gibt. (Dieses könnte man im Backend über die Settings definieren.) Wenn kein Bild aus keiner Quelle vorhanden ist, dann zeige eben den Bereich oben gar nicht an/baue dazu kein HTML-Code.

SebastianHonert commented 6 years ago

Habe ich wie folgt umgesetzt:

D.h. wir haben vier Varianten: Slide(s), Artikelbild, Default-Bild und abgeschaltet.

Es gibt aber noch ein Problem mit den Default-Settings. Die werden zwar nach Plugin-Aktivierung in den Einstellungen angezeigt, müssen aber erst einmal manuell gespeichert werden.

xwolfde commented 6 years ago

Jupp.

Allerdings fehlt jetzt bei einem Default-Rücksprung auf das Artikelbild der Seite die Anzeige des Titels (Seitentitel).

SebastianHonert commented 6 years ago

Stimmt. Dann wäre es vielleicht auch sinnvoll, eine Option anzubieten, ob bei Fallback zum Beitragsbild der Titel angezeigt werden soll, um auch Landing Pages ohne Titel im Slider anzulegen.

SebastianHonert commented 6 years ago

OK, habe ich so umgesetzt.

xwolfde commented 5 years ago

Ich glaub das kann zu, oder? (Ich hab Hemmungen , Issues in anderen Repos zu schließen - das steht nur dem jeweiligen Projektowner zu, in diesem Fall dir)