RRZE-Webteam / FAU-Einrichtungen

WordPress-Theme für zentrale Einrichtungen der Friedrich-Alexander-Universität Erlangen-Nürnberg
GNU General Public License v3.0
18 stars 11 forks source link

Hero in Header verschieben zur Standardkonformität zu WP #1223

Closed xwolfde closed 1 year ago

xwolfde commented 1 year ago

Der Hero-Bereich ist derzeit Teil des Contents, bzw. der Seitentemplates.

Da dies bei vielen Custom Post Types von Plugins Probleme macht und erzwingt dass man da dann für das Thema extra was machen muss, soll der Hero in der wp_head() wandern.

Dabei ist bei dem Ausrollen darauf zu achten, dass die gäbngigen RRZE Plugins, die eigene FAU-Templates beinhalten, dementsprechend angepasst werden, so daß dann keine doppelte Hero erscheint.

xwolfde commented 1 year ago

Möglicherweise betroffene Plugins mit FAU-Theme-spezifischen Templates, die angepasst werden müssten:

xwolfde commented 1 year ago

Im Theme umgesetzt. Die Plugins müssen noch angepasst werden.

Am Besten sollte in diesen eine Überprüfung der Version statt finden. Wenn <2.3 bei der Theme Version, dann sollen die ihren modifizierten FAU-Theme Template nutzen, ansonsten den von jeden Default-Theme.

Dies kann so aussehen:

get_header(); 
$currentTheme = wp_get_theme();     
$vers = $currentTheme->get( 'Version' );
  if (version_compare($vers, "2.3", '<')) {  
    // alte Anweisung für den Hero hier....
  }
cassandre commented 1 year ago

Bei CPT-Archiven wird der hero-small statt hero-index angezeigt: https://www.beta.wordpress.rrze.fau.de/room/. Kann es sein, dass in header-hero.php noch ein switch für is_archive() fehlt?

xwolfde commented 1 year ago

@cassandre Jein, das kann man in den Customizer einstellen, welcher Hero gezeigt wird. Unter Customizer-> Anzeigeoptionen->Header:

grafik

Dort ist gerade bei beta voreingestellt, das die Standardanzeige (die schmale) verwendet wird.

xwolfde commented 1 year ago

@cassandre Das Problem mit dem Title im Hero sollte auch mit der neuen Version behoben sein (konkret ab 2.3.8)

xwolfde commented 1 year ago

Sieht IMHO gut aus. Finde keine Probleme mehr