[Typo3] Überflüssigen Quellcode entfernen (jp_pageteaser)

[Nina]

Ich nutze die Extension "jp_pageteaser" und bin etwas über den Quelltext entsetzt, den diese Extension teilweise produziert. Daher würde ich den gerne säubern. Ich habe aber keinen Plan wo man in so einer Extension die Anweisungen für den Quellcode findet ...

Da Steffen (wohlverdient) im Urlaub ist, kann ich ihn nicht deswegen befragen. Hat von euch jemand Erfahrung damit und kann mir da weiterhelfen bzw. Tipps geben wo sich der böse Quellcode versteckt?

Danke

Problem gelöst. Danke an alle

[Nina]

Ich hab inzwischen rausgefunden, dass diese Infos in der angehangenen Datei sind. Zwei nervige DIVs habe ich bereits entfernt, aber nun stehe ich vor dem Problem, dass ich dem Bild gerne eine andere Klasse zuweisen würde. Es hat bereits eine Klasse namens "tx-jppageteaser-pi1-image", aber ich finde diese Angabe nirgends und kann sie daher nicht austauschen. Ne Idee wo die sein könnte?

[Nina]

Was mich auch irritiert ist, dass der Teaser standardmäßig den Text mit einem doppelten P umschließt. Komische Sache ...

<p><p>Beispieltext</p></p>

[Kafkaesk]

Hallo Nina,

die Extension bringt - wie die meisten - eine eigene HTML-Vorlage mit bzw. in dem Fall sogar mehrere, die sich irgendwie so im Constant des entsprechenden Templates eingebunden werden:

#Teaser
plugin.tx_jppageteaser_pi1{    
    maxWidth = 100    
    templatePath = fileadmin/templates/teaser/      
    defaultTemplateFileName = 1column_image_text.htm    
} 

Such doch mal diese Stelle, dann weißt Du zumindest wo diese HTML-Vorlage findest, damit Du sie entsprechend bearbeiten kannst. In dieser HTML-Vorlage kannst Du auch die CSS-Zuweisungen machen. Das ist dann völlig wurscht, ob Du die gegebenen übernimmst oder ganz eigene entwickelst.

Das Ganze sollte dann im Setup des entsprechenden Templates dann wahrscheinlich so ausschauen:

#Teaser
plugin.tx_jppageteaser_pi1 {
   _CSS_DEFAULT_STYLE >
   text.wrap =   
} 

Hier wird mit den folgenden Zeilen im Setup Folgendes bewirkt:

• Das Standard-CSS wird aus dem Head der generierten Einzelseite herausgenommen und kann dadurch einem externen CSS-File zugewiesen werden.
• Und text.wrap = bewirkt die Auflösung der standardmäßigen "Ummantelung" des p-Tags (was wohl dieses von Dir beschriebene Problem lösen dürfte.

Schau Dir auf jeden Fall folgende Dateien mal an:

• ext_typoscript_constants.txt
• ext_typoscript_setup.txt

Die befinden sich hier: typo3conf -> ext -> jp_pageteaser. In diesen beiden Dateien wird die ganze Konfiguration der Extension beispielhaft erklärt.

Solltest Du weiter Schwierigkeiten damit haben, kann ich gerne mal morgen nachschauen.

[Nina]

Danke Kafkaesk,

die Template-Datei habe ich gestern nach der Anweisung von Steffen kopiert und im fileadmin in den templates-Ordner gepackt (er hat die Zuweisung entsprechend angepasst). Ich hab an diesem template auch schon schön rumgebastelt, aber die jetzt von mir angesprochenen Probleme (doppeltes P, nicht zu findende Klassenzuweisung im Bild) stehen ja im Template nicht.

Derzeit sieht das Template bei mir so aus:

<!--- ###TEASER### begin --->
<div class="clearfix teaser">
   ###TEASER_HEADLINE###
   ###TEASER_IMAGE###
   ###TEASER_TEXT###
   <p class="detail"><img src="fileadmin/images/doppelpfeil2.gif" height="14" width="14" alt=""> ###LINK_ITEM###</p>
</div>
<!--- ###TEASER### end --->

Die zwei von dir genannten Dateien habe ich mir schon angeschaut, aber als Typo-Newb werde ich nicht schlau, was ich da wie ändern müsste. Wäre lieb wenn du mir da helfen könntest

[Kafkaesk]

Danke Kafkaesk,

die Template-Datei habe ich gestern nach der Anweisung von Steffen kopiert und im fileadmin in den templates-Ordner gepackt (er hat die Zuweisung entsprechend angepasst). Ich hab an diesem template auch schon schön rumgebastelt, aber die jetzt von mir angesprochenen Probleme (doppeltes P, nicht zu findende Klassenzuweisung im Bild) stehen ja im Template nicht.

Derzeit sieht das Template bei mir so aus:

<!--- ###TEASER### begin --->
<div class="clearfix teaser">
   ###TEASER_HEADLINE###
   ###TEASER_IMAGE###
   ###TEASER_TEXT###
   <p class="detail"><img src="fileadmin/images/doppelpfeil2.gif" height="14" width="14" alt=""> ###LINK_ITEM###</p>
</div>
<!--- ###TEASER### end --->

Die zwei von dir genannten Dateien habe ich mir schon angeschaut, aber als Typo-Newb werde ich nicht schlau, was ich da wie ändern müsste. Wäre lieb wenn du mir da helfen könntest

Gut, die Dateien kann ich mir ja morgen mal anschauen. Und was diese HTML-Vorlage angeht, siehst Du ja selbst, dass derzeit nur die Klassen "clearfix teaser" und "detail" definiert sind. Also mußt Du für das Bild erst noch eine Klasse einbinden. Versuch doch mal um ###TEASER_IMAGE### eine CSS-Klasse zuzuweisen. Das müßte eigentlich funktionieren.

[Nina]

Das geht auf jeden Fall, aber ich möchte ja möglichst sauberen Quellcode generieren. Da dem Bild sowieso schon eine Klasse zugewiesen wird (durch die Extension), macht es ja Sinn herauszufinden wo/wie diese Klasse zugewiesen wird und sie entsprechend auszutauschen.

Ich bin dahingehend auch schon ein bisschen weiter. Und zwar habe ich herausgefunden, dass in der angehangenen Datei zumindest ein Teil der Klassenzuweisung erstellt wird. Wenn man diesen Teil ändert, wird auch der erste Teil der Klasse geändert.

var $prefixId = "tx_jppageteaser_pi1";

bewirkt derzeit eine Klasse namens tx-jppageteaser-pi1-image

Ändere ich das z.B. so ab:

var $prefixId = "teaserpic";

stünde nun folgendes da: teaserpic-image

und wenn ich nun in dieser Zeile das "image" aus (''image) lösche:

$lconf['image.']['params'] = $this->pi_classParam('image');

steht da nur noch ein überflüssiges - in der Klasse.

Doof ist allerdings, dass rund um alle Teaser herum noch ein (nicht von mir eingestelltes) DIV ist, das dummerweise dann ebenfalls die "teaserpic"-Klasse annimmt und somit das Layout zerschießt. Also muss ich erstmal rausfinden wie ich dieses überflüssige DIV loswerde ^^

[Nina]

*grrr* Ich hab die Klasse im Stylesheet jetzt auf teaserpic- umbenannt. Weder konnte ich herausfinden wo das überflüssige DIV herkommt, das alle Teaser umrandet (also ein Gesamt <div class="teaserpic">, das vom Template gar nicht beeinflusst wird. Selbst wenn gar nichts im Template stünde, wird dieses DIV eingeblendet), noch konnte ich den "-" nach "teaserpic" entfernen. Ist nicht optimal, aber ich finde wirklich nicht mehr raus.

Was richtig hingegen richtig heftig stört, sind eben diese doppelten

<p><p>beim Teasertext</p></p>

Beispielseite:
http://www.ars-akustika.com/cms/index.php?id=54

[steffenk]

Hi Nina,

Du kannst einige Sachen über das Setup bestimmen, zumindest kannst Du Standardformatierungen löschen.
Was Dich so stört, sind "wraps" - d.h. Tags, die drumrum sind.

In der Datei ext_typoscript_setup.txt siehst Du Beispiele. Diese ist nicht aktiv, kann aber über das Setup mit eingebunden werden.

Du kannst folgendes probieren (ohne Gewehr)

plugin.tx_jppageteaser_pi1 {
  text.wrap >
  headline.wrap >
  image.wrap >
  image.ATagParams = class="meinteaserbild"
  text.wrap >
  link.wrap >
}

Ich habe auf die Schnelle noch die Klasse verändert und die divs entfernt.

Bei weiteren Fragen kann Dir Alexander sicher helfen.

[Nina]

Schäm dich Steffen ... was bist du denn noch hier unterwegs? Husch husch, ab in den Urlaub

Ich werde mal schauen ob ich rausfinde wie man diese ext_typoscript_setup.txt -Datei einbindet und wie ich dieses verflixte Extra-Div rund um das Gesamtplugin loswerde ...

[Nina]

Komisch .. wenn ich im Typo bei Templates / root / constant editor / plugin.pageteaser das Feld "Wrap around the text" aktiviere und leer lasse, macht er gar keine

<p></p>

mehr um den Text.

Lasse ich stattdessen das default

<p>|</p>

an, macht er sie wieder doppelt um den Text

[steffenk]

| steht für den text, also mach nur | (ohne p's)

so, jetzt bin ich weg

[Nina]

@Steffen: Ich glaube, nächstes Mal müssen wir dich auf ner einsamen Insel absetzen wo weit und breit kein W-Lan ist

| steht für den text, also mach nur | (ohne p's)

Wenn ich es ohne p's mache, wird der Text überhaupt nicht mehr gestyled. Kaum gebe ich es wieder an, ist er wieder mit doppelten p's gestyled. Ich lass es jetzt aber erstmal ungestyled und schreibe dann einfach erstmal ins Template die p's fix außen rum.

Ist nicht sauber, aber jetzt zeigt er brav nur noch ein P an.
Problem gelöst (wenn auch unkonventionell ). Vielen Dank an alle Helfer