[3.2] [3.3] Collapse Quote

[hackepeter13]

Kann ich bestätigen, funktioniert jetzt alles soweit fehlerfrei.

>Edit:
Nicht zwingend notwendig, aber ich würde die Felder für die Farbcodes 8-Stellig, statt 6-Stellig machen, so kann man auch noch die Transparenz der Farbe einstellen.

[Kirk]

Oder man regelt dies über css, wäre insofern von Vorteil wenn man helle und dunkle Styles inst. hat. Damit könnte man es explizit auf den jeweiligen Style anpassen. Der Nachteil ist wenn jemand sich in Sachen css nicht auskennt.

[SeewolfPK]

Version 1.3.1 verfügbar

Der Link für zu Github Version 1.21, die bei mir im ACP auch noch als aktuell angezeigt wird.
Was läuft da falsch?

[IMC]

Hi Paul,

die Releases habe ich gelöscht da sie leider fehlerbehaftet waren.
Siehe: viewtopic.php?p=1426243#p1426243

Die Version 1.3.2 werde ich voraussichtlich in 2 Tage online stellen. Sie existiert im Moment als Beta2.

[IMC]

Version 1.3.2 verfügbar

[Dref]

Mir ist dieser Fehler gerade bei Version 1.3.2 aufgefallen

Code: Alles auswählen

Uncaught SyntaxError: redeclaration of let imcgerQuoteBox
    <anonymous> ./ext/imcger/collapsequote/styles/all/template/quote.js?assets_version=4989:1
quote.js:1:1
    <anonyme> ./ext/imcger/collapsequote/styles/all/template/quote.js?assets_version=4989:1
JQMIGRATE: $(html) HTML strings must start with '<' character jquery-migrate-1.2.1.js:41:12

Dieser Fehler wurde bei Verwendung des Inspektionstools angezeigt.

[Kirk]

Dies kann ich bei mir nicht nachvollziehen. Wie sehen deine Einstellungen dazu aus?
Thorsten
Mir ist aufgefallen das du im Controller die min. Sichtbare Textzeilen auf 4 begrenzt hast aber in der HTML hast du min="2" eingetragen.
Ist etwas verwirrend wenn z.B. nur 2 Zeilen haben möchte und nach dem speichern der Einstellung wieder 4 Zeilen drin stehen.
Diesen Code dafür im Controller bräuchte man gar nicht, setze doch den gewollten min. Wert in den HTML Dateien auf 4.

[IMC]

Ich konnte den Fehler bei mir nicht nachstellen.
Auch bin ich bin ich etwas irritiert. imcgerQuoteBox ist eine Klasse. Diese wird im Code nicht mit let deklariert.

Ist das Forum, in dem der Fehler auftritt, öffentlich zugänglich? Dann würde ich mir den Fehler dort etwas näher ansehen.

[IMC]

Mir ist aufgefallen das du im Controller die min. Sichtbare Textzeilen auf 4 begrenzt hast aber in der HTML hast du min="2" eingetragen.

Danke für den Hinweis, Das bereinige ich in der nächsten Version.

[LukeWCS]

Kann den Fehler auch nicht reproduzieren.

Auch bin ich bin ich etwas irritiert. imcgerQuoteBox ist eine Klasse. Diese wird im Code nicht mit let deklariert.

Das irritiert mich ebenfalls.

Apropos, dadurch gerade zufällig bemerkt, dass CQ kompatibel zu TC ist. Sowas gerne im TC Thema melden, weil in deinen Changelogs nichts dazu steht. Habs jetzt bei Position 16 in der Liste eingefügt, das gibts seit 1.3.0 wie ich sah.