FriendsOfREDAXO / base_quality_check

REDAXO 5 AddOn.
MIT License
5 stars 1 forks source link

Editor-Bindung an CKE5 muss überprüft werden #18

Open christophboecker opened 4 months ago

christophboecker commented 4 months ago

Bei mir sehen die Textarea im Formular für die Checks echt blöd aus:

grafik

In anderen Tabellen werden mir Textareas korrekt dargestellt. Die Ursache liegt in der festen Verdrahtung des Cke5-Editors.

grafik

Es gibt jetzt m.E. vier Möglichkeiten:

  1. Problem ignorieren
  2. keine Editor-Bindung, einfach nur Standard-Textarea
  3. CKE5-Editor-Addon in der package.yml unter requires anfordern, dann aber auch das Profiel mitliefern und installieren/deinstallieren.
  4. Maximale Flexibilität, indem der Editor individuell und updatesicher konfiguriert werden kann (z.B.in der System-Page was einhängen) oder über eine data/base_quality_check/config.yml oder einen Eintrag in der core/config.yml
christophboecker commented 3 months ago

21 setzt bezogen auf die Lösungsvorschläge oben eine Variante 5 um: lassen wie es ist mit Fallback auf einfache Textarea.

olien commented 3 months ago

Find ich so super und könnte so bleiben!

alxndr-w commented 3 months ago

Es fehlte form-control in den Klassen.

Ich löse das immer so:

https://github.com/alexplusde/events/blob/88f15883b3a559a12ce5165696286f1d78166f78/install/rex_event.tableset.json#L186

https://github.com/alexplusde/events/blob/88f15883b3a559a12ce5165696286f1d78166f78/boot.php#L3-L9

christophboecker commented 3 months ago

@alxndr-w :

1) ja, stimmt, hatte ich (mal wieder) nicht mehr auf dem Radar, dass das Hinzufügen der Klasse form-control eigentlich schon reicht.

2) Was Deine Lösung anbelangt: jeder hat so seine Tricks und Vorlieben, ohne eine davon als besser oder schlechter, richtiger oder falscher werten zu wollen. Ich baue halt ungerne Code am Ende um (also EP OUTPUT_FILTER), sondern lieber so früh wie möglich, also z.B. über mydataset::getForm. :-)

alxndr-w commented 3 months ago

mydataset::getForm() funktioniert leider nicht im klassischen Tabellen-Kontext.

christophboecker commented 3 months ago

mydataset::getForm() funktioniert leider nicht im klassischen Tabellen-Kontext.

Das ist zweifelsfrei richtig und der Grund, warum ich Tabellen mittlerweile immer eine eigene Klasse spendiere selbst wenn sie anfangs nichts macht außer zu existieren.

Mir fällt kein Szenario ein, in dem ich einerseits einem Feld Attribute und den zugehörigen Output-Filter verpassen kann aber nicht zugleich die Möglichkeit habe, eine Modell-Class zu setzen. Was habe ich übersehen? Nur mal so gefragt um schlauer zu werden.

alxndr-w commented 3 months ago

Ich meinte die Bearbeitungs-Ansicht im Table Manager, in der ich gelegentlich lande. Da möchte ich, dass die Bearbeitung genauso funktioniert wie in der Addon-Ansicht.

christophboecker commented 3 months ago

Nu hast Du mich verwirrt! Welche "Bearbeitungs-Ansicht im Table Manager"? Auch der Table-Manager berücksichtigt die zugeordnete Model-Class; das ist tief im System verankert. In diesem Addon hier wird die Tabelle derzeit nur im Table-Manager bearbeitet und die Änderung über die Model-Class ist voll wirksam. Gibt es irgendwo Ecken, die Formulare ohne Berücksitigung der Model-Class anzeigen? Das wäre m.E. ein Bug.

alxndr-w commented 3 months ago

🤔 Lass uns mal ne Videokonferenz machen oder in Slack weiterschreiben, du scheinst was zu wissen was ich noch nicht wusste.