Teaserbild wird zum Detail-Link

[neelix]

Hallo zusammen,

ich habe hier folgenden Effekt:

I.d.R. gibt es zu den Events keinen weiteren Inhalt. Es gibt also den Titel, meist ein Datum (ggf. mit Uhrzeit) und manchmal einen Ort. In den meisten Fällen gibt es zum Zeitpunk der Veröffentlichung auch nur den Titel, Zeit und Ort stehen dann noch nicht zwingend fest.

Soweit ist alles noch OK. Packe ich jetzt aber ein Bild dazu, dann bekommt das Bild einen Link zur Detailseite des Termins. Es scheint aber kein "Weiterlesen"-Button, da es ja keinen weiteren Inhalt gibt. Wie werde ich diesen Link los? Klick man nämlich drauf, landet man auf einer quasi leeren Seite, die nur die Daten aus dem Teaser (inkl. Bild) enthält. Und das sieht blöd aus.

Beispiel: https://dev.luetteraten.de/

Danke & Gruß

Neelix

[planepix]

Schau mal hier Neelix:
https://community.contao.org/de/show...h-ohne-Details

[neelix]

das klappt noch. Wunderbar.

Allerdings frage ich mich gerade, warum das nicht per Default so ist. Was hat sich der Künstler dabei gedacht?

[Spooky]

Alle Events und Nachrichten mit Weiterleitungsziel "Standard" haben eine Detail URL. Wenn kein dedizierter Detail Inhalt vorliegt, geben die Standard Templates Teaserbild und -text aus.

[neelix]

das klappt noch. Wunderbar.

Klappt leider doch nicht so ganz. https://community.contao.org/de/show...l=1#post571758

[neelix]

Hi,

der Thread ist schon etwas älter, aber ich versuche mal mein Glück :-)

Ich muss, genau wie der TE, den Link auf dem Bild loswerden und bin der Empfehlung gefolgt:

Das ...

<?php $this->insert('image', $this->arrData); ?>

... ersetzen durch das:

<?php if ($this->hasDetails): ?>
   <a href="<?= $this->href ?>" title="<?= $this->readMore ?>"<?= $this->target ?> itemprop="url">
      <img src="<?php echo $this->src ?>" class="<?php echo $this->floatClass ?>">
   </a>
<?php else: ?>
   <img src="<?php echo $this->src ?>" class="<?php echo $this->floatClass ?>">
<?php endif; ?>

Das hat den gewünschten Effekt - der Link ist weg. Aber leider ist nun auch die "figCaption"/ Bildunterschrift weg. Wie kann ich die wieder reinschreiben?

Danke!!

Ich habe diese Änderungen im Template gemacht. Damit ist dann das Verhalten weg, dass das Bild zum Link wird.

Aber die Änderungen aus der Zeit von Contao 4.4 berücksichtigen die "responsive Images"-Funktion nicht. Es wird also kein Src-Set mehr definiert sondern nur ein IMG eingebunden.
Wie bekomme ich das jetzt wieder rein?

[Spooky]

Generell musst du

<?php $this->insert('image', $this->arrData); ?>

verwenden.

Um welches Modul genau geht es überhaupt? Wenn du die Verlinkung im Nachrichten- oder Eventleser-Modul meinst: das wird in einer der nächsten Contao Versionen behoben sein (siehe https://github.com/contao/contao/pull/5851). Bis dorthin kannst du in deinem Template einfach

<?php $this->href = null; ?>

einfügen.

[neelix]

Ja, Nachrichten und Events.

Der Block lautet ja im Original:

   <?php if ($this->addImage): ?>
     <?php $this->insert('image', $this->arrData); ?>
   <?php endif; ?>

und erzeugt automatisch den Link.

Wo soll denn dein Code hin?

[Spooky]

Irgendwann vor

<?php $this->insert('image', $this->arrData); ?>

[neelix]

Also z.B. so:

   <?php if ($this->addImage): ?>
     <?php $this->href = null; ?> 
     <?php $this->insert('image', $this->arrData); ?>
   <?php endif; ?>

Hat aber keine Auswirkung.

[Spooky]

Sorry,

<?php $this->href = $this->imageHref = null; ?>

muss es sein.

[neelix]

So, passt es.

Kommt die Anpassunge auch für die 4.13 oder nur in der 5.x?

[Spooky]

Der PR ist für 4.13

[neelix]

Wie verhält sich denn die 5er an der Stelle, wäre ja blöd, wenn das ursprüngliche Verhalten da wieder drin wäre.

[Spooky]

Wie meinst du? Das Verhalten ist aktuell in 4 und 5 gleich. Der PR ist noch nicht gemerged und daher auch noch nicht als neue Contao Version veröffentlicht.

[neelix]

Ich stelle gerade fest: Der hat eine blöde Nebenwirkung:

Der Link unter "Weiterlesen" bei den Terminen mit Inhalt hat dann kein Ziel mehr.

https://dev.luetteraten.de/

an dem zweiten pinken Block.

[Spooky]

Poste dein Template.

[neelix]

<?php if ($this->header): ?>
  <div class="header<?= $this->classHeader ?>">
    <?php if ($this->firstDay): ?><span class="day"><?= $this->firstDay ?>, </span><?php endif; ?><span class="date"><?= $this->firstDate ?></span>
  </div>
<?php endif; ?>

<div class="event layout_teaser<?= $this->classList ?>">

  <?php if ($this->hasDetails): ?>
    <h2><a href="<?= $this->href ?>" title="<?= $this->title ?> (<?php if ($this->day): ?><?= $this->day ?>, <?php endif; ?><?= $this->date ?><?php if ($this->time): ?> <?= $this->time ?><?php endif; ?>)"<?= $this->target ?>><?= $this->link ?></a></h2>
  <?php else: ?>
    <h2><?= $this->title ?></h2>
  <?php endif; ?>

  <p class="time"><time datetime="<?= $this->datetime ?>"><?= $this->date ?><?php if ($this->time):?> <?= $this->time ?><?php endif; ?></time></p>

  <?php if ($this->location): ?>
    <p class="location"><?= $this->location ?><?php if ($this->address): ?> <br> <?= $this->address ?><?php endif; ?>
  <?php endif; ?>

  <div class="ce_text block">
    <?php if (!$this->addBefore): ?>
      <?= $this->teaser ?>
    <?php endif; ?>

   <?php if ($this->addImage): ?>
      <?php $this->href = $this->imageHref = null; ?> 
     <?php $this->insert('image', $this->arrData); ?>
   <?php endif; ?>

    <?php if ($this->addBefore): ?>
      <?= $this->teaser ?>
    <?php endif; ?>
  </div>

  <?php if ($this->hasDetails): ?>
    <p class="more"><a href="<?= $this->href ?>" title="<?= $this->readMore ?>"<?= $this->target ?>><?= $this->more ?><span class="invisible"> <?= $this->link ?></span></a></p>
  <?php endif; ?>

</div>

<?php

// This template is used as an event list template by default, so we only add
// JSON-LD data in case this is an event without a reader
if (!$this->hasReader) {
    $this->addSchemaOrg($this->getSchemaOrgData());
}

[Spooky]

Bei den Events ist es leider so, dass der Link nur in href drin steht - welcher auch als Bild Link benutzt wird. Für Events müsstet du den Bild Link daher eher so entfernen:

<?php $this->insert('image', array_merge($this->arrData, ['href' => null, 'imageHref' => null])) ?>

(ungetestet)

[neelix]

Das sieht gut aus.
Jetzt haben die Bilder zwar auch bei "Weiterlesen..." Link keine Verlinkung mehr, das ist aber nicht tragisch.

[Andreas]

Wahrscheinlich hast du die Bedingung vergessen, wann die Variablen auf NULL gesetzt werden sollen.

<?php if ($this->hasDetails): ?>

Ich hatte das letztens in Contao 4.9 ohne Templateanpassung mit JS erledigt.

<!-- templates/j_misc.html5 -->

<script>
  jQuery(function($) {

    /* Deactivate links from images in news and events if not needed (Contao bug https://github.com/contao/contao/issues/5477) */
    var listItems = $(':is(.mod_eventlist, .mod_newslist) > div');

    listItems.each(function(i, ele) {
      ele = $(ele);
      var hasMore = ele.find('.more').length > 0 ? true : false;
      if(!hasMore) {
        ele.find('img').prependTo(ele.find('figure'));
      }
    });

  });
</script> 

[aitana]

Hallo zusammen

Das Problem mit der automatischen Verlinkung der Titel und Bilder ohne Detailseite scheint nach wie vor ein Problem zu sein :-)
Ich konnte das mit der hier geposteten Template Anpassung lösen. Aber wäre es nicht gut, wenn das direkt im Standard-Template so wäre?

Was mir noch aufgefallen ist:
In dem Moment, wo Text im Detail Inhaltselement eingetragen ist, wird der Teasertext überschrieben.
Der Detailtext erscheint also als Teaser und als Inhaltstext.
Vielleicht übersehe ich aber auch irgendwelche Einstellungen.

[Spooky]

Aber wäre es nicht gut, wenn das direkt im Standard-Template so wäre?

Das wurde explizit mal so geändert, damit das Teaser Bild immer verlinkt ist - weil man das meistens haben will.