Twig Template "player"

[herr rilke]

hallo,

nach dem update auf C 5.3 funktioniert mein altes ce_player template nicht mehr.

ich möche aus dem output den

<div class="content-player" ...>

entfernen. wie kann ich das im twig-template machen (oder wär's einfacher legacy content elements zu "restoren")?
da bin ich völlig lost

{% extends "@Contao/content_element/player.html.twig" %}

{#
  ** Add changes to the base template here. **

  Hint: Try adjusting blocks and attributes instead of
  overwriting the whole template. This way your version
  can remain compatible with future changes to the base
  template as well as adjustments made by extensions.

  Currently available blocks:
    "picture_component", "image", "sources", "source",
    "schema_org", "figure_component", "media",
    "media_link", "caption", "caption_inner",
    "content", "editor_view", "metadata",
    "headline_component", "headline_attributes",
    "headline_inner", "wrapper", "wrapper_tag",
    "attributes", "inner", "style", "script"

  Example:
    {% block picture_component %}
       {{ parent() }}
       My additional content for 'picture_component'…
    {% endblock %}
#}

[Spooky]

Warum stört dich die Klasse?

[herr rilke]

ich lade videos per inserttag in ein anderes element und da zerlegt's mir das layout

[mlweb]

Dann solltest Du das CSS für diese Klasse suchen und löschen oder verändern. Sehe ich als deutlich einfachere Lösung an.

[herr rilke]

Dann solltest Du das CSS für diese Klasse suchen und löschen oder verändern. Sehe ich als deutlich einfachere Lösung an.

ja, 100 pro.

allerdings passiert folgendes:

input (RTE):

<p style="text-align: center;"><em>"Einleitung...Text..."</em><br><br>{{insert_content::729}}</p>

wird von contao zu:

 
	<p style="text-align: center;">
                    <em>"Einleitung...Text..."</em><br><br>
        </p>

<div class="content-player">            
                                <figure>
                                    <video controls="" poster="....jpg" width="260" height="195" preload="metadata">
                                         <source type="video/mp4" src="....mp4">
                                   </video>
        
                                 </figure>

            </div>


der <p> wird automatisch geschlossen und dann erst der div.content-player geöffnet
                 

[Spooky]

Du kannst nicht innerhalb eines <p> ein {{insert_content::*}} nutzen. Das wird in den meisten Fällen zu invalidem HTML Code führen. Die CSS Klasse ist nicht das Problem.

[herr rilke]

sieht ganz so aus.
allerdings gings in C4.13 noch.

wie dem auch sei: auch der <p> wird ja vom RTE / von contao gesetzt.
muss ich mir jetzt ein custom element schreiben oder gibt's eine andere lösung?

[Spooky]

allerdings gings in C4.13 noch.

Hier gibt es keinen Unterschied zwischen den Contao Versionen. Das Problem wirst du immer haben.

[herr rilke]

hab' n live-beispiel, wo's jetzt nen paar jahre gelaufen ist:

https://dgwf.net/stimmen.html

[Spooky]

hab' n live-beispiel, wo's jetzt nen paar jahre gelaufen ist:

https://dgwf.net/stimmen.html

Da lässt sich nicht ablesen, wo du da so einen Insert-Tag eingesetzt hättest (das ist ja nur dir bekannt). Unabhängig davon hat diese Seite einige Fehler, die du beheben solltest: https://validator.w3.org/nu/?doc=htt...2Fstimmen.html

oder gibt's eine andere lösung?

Du kannst den HTML-Editor des TinyMCE nutzen und

<div>{{insert_content::*}}</div>

schreiben (natürlich außerhalb von <p>…</p>).

[herr rilke]

hi spooky,

den insert-tag habe ich so gesetzt wie ganz oben beschrieben.

für mich ist die HTML variante eine möglichkeit, für die redakteure sicherlich nicht.

heißt also für mich unterm strich:

• über das twig template komme ich da nicht ohne weiteres dran
• ich schreibe ein custom element, wenn's endbenutzerfreundlich sein muss

OT: wie geht ihr mit solchen zusatz-aufwänden um, wenn also durch ein update plötzlich und unerwartet malessen entstehen?

[Spooky]

den insert-tag habe ich so gesetzt wie ganz oben beschrieben.

Das kann nicht sein. In der von dir verlinkten Installation hast du dann evt. noch eine zusätzliche Template Anpassung für die jeweiligen Inhaltselemente, die du per Insert-Tag eingebunden hast.

[mlweb]

Für mich kommt es immer drauf an. Hätte ich das von Anfang an besser oder anders lösen können bzw. sollen, dann ist das mein Bier - ohne wenn und aber.
Ein {{insert_content::*}} innerhalb von p-Tags war ja noch nie richtig (auch wenn es bei Dir bisher kein Layoutproblem gegeben hat), wäre also mein Bier.

Ansonsten für Kunden, die Updateservive haben, richte ich auch alle Sachen mit, die im Rahmen eines Updates notwendig sind, das sind dann eben auch erforderliche Template-und Layout-Anpassungen etc. Die Pauschale für den Updateservice muß eben so kalkuliert sein, dass ich im Durchschnitt nicht zusetzte, aber auch so, dass ich den Kunden nicht über den Tisch ziehe. Erfordert etwas Erfahrung, aber das hat man m.E. relativ schnell im Griff. Außerdem sind Major-Updates (also Update 4 auf 5) mit höherem Anpassungsaufwand außen vor und werden individuell angeboten.

Bei Einzelupdates, wird bei mir klar nach Aufwand abgerechnet und ich achte außerdem darauf, dass ein Kunde ohne Updateservice abrechnungstechnisch nicht besser wegkommt als ein Kunde mit Updateservice.

[herr rilke]

danke für deine einschätzung, mlweb!

[Spooky]

Ich denke ich habe deine ursprüngliche Anfrage falsch verstanden. Du willst nicht die CSS-Klasse entfernen, sondern den gesamten Wrapper, korrekt?

Hier musst du grundsätzlich genau so vorgehen wie in deiner Contao 4 Installation. Du darfst nicht vom _base Template extenden, sondern überschreibst das gesamte player Template komplett. Bspw.

{# templates/content_element/player/without-wrapper.html.twig #}
<{{ figure.media.type }}{{ figure.media.attributes }}>
    {% for source_attributes in figure.media.sources %}
        {% do csp_source('media-src', source_attributes.src) %}
        <source{{ source_attributes }}>
    {% endfor %}
</{{ figure.media.type }}>

{% for source_file in source_files %}
    {% do add_schema_org(source_file.schemaOrgData|default) %}
{% endfor %}

(ungetestet)

[herr rilke]

ja, genau, der ganze <div> soll weg!

danke für's nachlesen, erläutern und schreiben!

klappt 1a ... und tut, was es soll. funktioniert auch (wieder) im output!


danke dir.



OT: mir macht's klar, dass ich auf solche lösungen nicht von allein kommen werde; mir graust es vor der umstellung aller templates auf twig ...

[mlweb]

Ok, dann hatte ich es auch falsch verstanden und dachte es soll nur die Klasse entfernt werden.

[herr rilke]

danke für die rückmeldung.
versuche demnächst, noch genauer zu sein