Twig Template "player"

hallo,

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

ich möche aus dem output den

HTML-Code:

---

<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

HTML-Code:

---

{% 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 %}
#}

---

Warum stört dich die Klasse?

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

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

Zitat:

---

Zitat von 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.

---

ja, 100 pro.

allerdings passiert folgendes:

input (RTE):

HTML-Code:

---

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

---

wird von contao zu:

HTML-Code:

---


        <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
               

---

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.

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?

Zitat:

---

Zitat von herr rilke

allerdings gings in C4.13 noch.

---

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

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

https://dgwf.net/stimmen.html

Zitat:

---

Zitat von herr rilke

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

Zitat:

---

Zitat von herr rilke

oder gibt's eine andere lösung?

---

Du kannst den HTML-Editor des TinyMCE nutzen und

HTML-Code:

---

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

---

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

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?

Zitat:

---

Zitat von herr rilke

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.

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.

danke für deine einschätzung, mlweb!

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.

HTML-Code:

---

{# 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)

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 ...

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

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