Refactor: Renderpfade schrittweise vereinheitlichen (Parser + Flex-Repeater)

[skerbis]

Hintergrund

Aktuell existieren in MForm mehrere Renderpfade mit teilweise überlappender Layout-Logik (u. a. Wrapper/Row/Column/Modal in klassischem Parser und Flex-Repeater).

Das ist historisch nachvollziehbar, führt aber in der Praxis zu:

• doppelter Pflege bei Layout-Fixes
• erhöhtem Risiko für Verhaltensdrift
• wiederkehrenden Regressionen (z. B. Row/Col-Themen)

Ziel

Schrittweise Vereinheitlichung der Renderlogik ohne Breaking Changes für bestehende Module.

Vorschlag (inkrementell)

1. Gemeinsamen internen Layout-Core definieren

• Zentrale Regeln für Wrapper-Strukturen (Row, Column, Modal, Tab, Collapse).
• Parser und Flex-Repeater nutzen dieselben Kernregeln.

2. Bestehende öffentliche APIs unverändert lassen

• Keine Änderung an add*()-Methoden-Signaturen.
• Keine Änderung an gespeicherten Datenformaten.

3. Kompatibilitätsnetz einziehen

• Snapshot-/Golden-Tests für typische Wrapper-Kombinationen.
• Spezifische Regression-Tests für addColumnElement + Repeater + Modal.

4. Optionales Verhalten über Attribute statt Sonderpfade

• Erweiterungen über klar benannte Attribute/Klassen.
• Kein implizites Hart-Coding pro Pfad.

5. Migration in kleinen PR-Schritten

• Erst intern konsolidieren, dann sukzessive alte Duplikate abbauen.
• Jede Stufe mit php-cs-fixer + rexstan + visueller Gegenprobe absichern.

Akzeptanzkriterien

• Gleiches Verhalten in klassischem Parser und Flex-Repeater für identische Wrapper-Szenarien.
• Keine erforderlichen Anpassungen an bestehenden Modulcodes.
• Dokumentierte, optionale Erweiterungspunkte (Attribute/Klassen).
• Kein Rückschritt in den bestehenden Row/Col-Fixes.

Notizen

• Der aktuelle Stand enthält bereits Fixes für Row/Col-Probleme.
• Dieses Issue dient als struktureller Folgepunkt, um künftige Doppel-Fixes zu vermeiden.

[caesarvoelkin]

Weiss nicht, ob das hier hin gehört: Sind nebst Layoutfixes (an strukturellen Elementen) auch Designfixes angedacht? Resp. Layoutfixes für Input-Felder? Mir ist aufgefallen, dass es diverse Unterschiede in den Eingabefeldern gibt (Schatten, Linien, Light vs Darkmode etc., Borderradius). Weiter möchte bei mir weder ->setFull() noch ['full' => true] funktionieren. – Ansonsten sehr cool, dieser neue Repeater! <3

[skerbis]

bitte neues issue dafür .. danke

[skerbis]

Update zum Stand der Vereinheitlichung (Parser + Flex-Repeater)

Erledigt

• Gemeinsamer Label-/Tooltip-Renderer eingeführt und in beiden Pfaden aktiv (MFormLabelRenderer).
• Gemeinsamer Layout-Core für Row-Klassen eingeführt (MFormLayoutCore):
  • Columns: data-group-column-row-class / data-group-row-class
  • Modal: data-modal-row-class / data-group-row-class
• setFull() / full => true im Flex-Repeater auf Parser-Verhalten angeglichen (volle Label-/Field-Spaltenbreite).
• Renderer-Parität als Smoke-Check-Demo eingebaut (demo_renderer_parity) inkl. Vergleich Parser-HTML vs. Repeater-Template-HTML.
• Dokumentation/Changelog ergänzt (inkl. Tooltip-Default-Icon fa-info-circle ohne Icon-Override).

Aktuell verifiziert

• Paritätsseite zeigt die Kernmarker in beiden Pfaden konsistent (Tooltip, Row-/Modal-Klassen, Full-Layout).
• Keine API- oder Datenformatänderung erforderlich.

Soll als Nächstes folgen

• Weitere strukturelle Regeln in den gemeinsamen Core ziehen (insb. Tab-/Collapse-/Accordion-Sonderfälle), um verbleibende Duplikate abzubauen.
• Paritätsnetz erweitern: zusätzliche Szenarien/Marker für komplexe Wrapper-Kombinationen (Tabs + Collapse + Modal + Repeater).
• Optional: automatisierte Snapshot-/Golden-Checks ergänzen, damit Drift früh CI-seitig sichtbar wird.
• Begleitend: Design-/Styling-Themen getrennt weiterführen (eigene Schiene/Issue), damit strukturelle Vereinheitlichung sauber abgegrenzt bleibt.

Zielbild bleibt unverändert

• Schrittweise interne Konsolidierung ohne Breaking Changes bei bestehendem Modulcode.

[skerbis]

Kurzes Follow-up: Was als Nächstes noch getan werden sollte

1. Tab-Logik final intern entkoppeln (ohne Tooling-Risiko)

• Die in beiden Pfaden bereits angeglichene Tab-Logik (active/nav/layout/style + Truthy-Flags) in einen stabilen gemeinsamen internen Helper überführen.
• Wichtig: In einem separaten kleinen Schritt, damit statische Tooling-Meldungen sauber nachvollziehbar bleiben.

2. Paritätsnetz weiter ausbauen

• Auf der Renderer-Paritätsseite zusätzliche Marker-Checks für komplexere Tab/Cascade-Szenarien ergänzen:
  • Tabs in Collapse
  • Tabs im Modal
  • Verschachtelte Tabs in Repeater-Items
• Ziel: Drift früh sichtbar machen, bevor es in realen Modulen auffällt.

3. Attribut-Serialisierung gezielt harmonisieren

• Parser/Flex nutzen aktuell noch getrennte Attribut-Renderer-Methoden.
• Nächster Schritt: gemeinsame Semantik definieren (welche Keys raus, welche escaped), dann schrittweise zusammenführen.

4. Wrapper-Sonderfälle reduzieren

• Nach Columns/Modal/Collapse als nächstes tab-/accordion-nahe Sonderpfade zusammenziehen.
• Fokus auf "gleiche Entscheidung, gleicher Output" statt auf große Umbauten.

5. Sicherheitsschiene für Änderungen

• Bei jedem Schritt: php -l + RexStan + Paritätsseite als Pflichtcheck.
• Änderungen weiterhin klein halten (inkrementelle PRs), um Regressionen schnell einzugrenzen.

Damit bleibt die Strategie "schrittweise intern konsolidieren, ohne Breaking Changes" erhalten, aber mit klarer nächster Arbeitsreihenfolge.