Comments

Joachim Namyslo created an issue. See original summary.

joachim namyslo’s picture

joachim namyslo’s picture

Status: Needs work » Needs review

Soweit mit Vorschlägen versehen. Die müssen allerdings noch reviwed werden, da das Wording an einigen Stellen noch der Kontrolle und der Nachbesserung bedarf. Die nicht übersetzen Zeichenfolgen können nicht übersetzt werden, da ein Fehler im Core dazu führt, dass der Mallformed HTML Filter ein Warnung beim Import der Datei angibt, wenn wir das tun.

Anders ausgedrückt. HTML-Elemente wie Button gehören einfach nicht in eine Orignalzeichenfolge. Mitgeteilt habe ich das dem Maintainer des Moduls bereits, aber offenbar hat er mich nicht verstanden oder verstehen wollen.

joachim namyslo’s picture

joachim namyslo’s picture

joachim namyslo’s picture

joachim namyslo’s picture

Hier bracht es einen Patch, sonst kriegen wir die restlichen Strings nicht weg und das Modul nicht auf 100 % Entweder müssen die CSS-Klassen aus den Strings oder wir müssen den Core Patcvhen,. damit er das Zulässt, was wegen des XSS-Scripting-Filters nicht einfach werden wird, Also müssen wir das Modul zerpflücken und die t()-Funktion verschieben. Dem Maintainer scheint das aber egal zu sein. Wir haben Ihn mehrfach darauf hingewiesen, Vielleicht hat er es auch einfach nicht richtig verstanden.

Sowas wie <button type="button" data-role="none" class="slick-prev">Previous</button> Kann der Filter nicht leiden und wirft den String als ungültiges HTML aus. Die funktion t() muss also um das Wort Previous und nicht um den Button-Tag. Wenn das jemand von euch hinbekommt, können wir den Issue zu machen und vertgessen, bis das Modul ein UI-relevantes Update bekommt.

joachim namyslo’s picture

joachim namyslo’s picture

Status: Needs review » Fixed
joachim namyslo’s picture

Status: Fixed » Needs work
joachim namyslo’s picture

Status: Needs work » Fixed
joachim namyslo’s picture

Status: Fixed » Closed (fixed)