Skip to content

Screenshots im README, JSON-LD-Vorschau repariert (v2.47.3) - #152

Merged
daimpad merged 1 commit into
mainfrom
claude/fix-data-prep-errors-kJYpl
Nov 6, 2026
Merged

daimpad merged 1 commit into
mainfrom
claude/fix-data-prep-errors-kJYpl

Conversation

@daimpad

@daimpad daimpad commented Nov 6, 2026

Copy link
Copy Markdown
Owner

Worum es geht

Das README bekommt Screenshots. Aufgenommen sind sie in einem lokalen, deutschen WordPress mit den 42 mitgelieferten Beispieldatensätzen. Dabei fielen drei Fehler auf, die dieser PR mit behebt.

Screenshots (docs/screenshots/, nicht im Plugin-ZIP)

Datei Platz im README
formular.png, katalog.png Überblick direkt unter der Einleitung
datensatzliste.png Datensätze verwalten
vorschau.png Geführter Wizard
batch-import.png Batch-Import (Vorschau mit einer fehlerhaften Zeile)
qualitaet.png Qualitätsindikatoren
katalog-detail.png Datenkatalog
katalog-filter.png Filter und Suche

Alle Aufnahmen sind in doppelter Auflösung. verify-package.py bestätigt, dass docs/ nicht im ZIP landet.

Fehlerbehebungen

  • Tab 5 „Vorschau“ zeigte nie das JSON-LD. get_preview_html() lief schon bei carbon_fields_register_fields. Zu diesem Zeitpunkt liefert get_the_ID() noch false. Unter dem Hinweis „Die JSON-LD-Ansicht unten …“ standen deshalb weder Code noch der Button „REST-Endpoint öffnen“. set_html() erhält jetzt einen Callback, den Carbon Fields erst beim Rendern aufruft.
  • Lange JSON-LD-Zeilen spannten das Formular über den Rand. Das <fieldset> von Carbon Fields hat ab Werk min-width: min-content. Gesetzt ist jetzt min-width: 0 auf fieldset, .cf-container__fields und die Felder darin. Bei 1280 px laufen alle fünf Reiter ohne horizontales Scrollen.
  • Die Live-Vorschau zeigte Rohwerte. Beim Thema stand die EU-URI statt „Umwelt“, bei der Dateigröße die Bytezahl. Jetzt erscheinen die Beschriftungen der Mehrfachauswahl und die Größe wie auf der Download-Card. Beim Reiterwechsel wird neu gelesen, weil die Mehrfachauswahl kein input/change auslöst.
  • Beispieldaten: In zwei Datensätzen stand der HVD-Code c_b79e35eb als Schlagwort. Jetzt steht dort „Mobilität“, in CSV und JSON.

Tests

  • PHPUnit: 505 Tests grün. Neu ist eine Prüfung, dass in den Beispieldateien kein Vokabular-Code als Schlagwort steht. Sie schlägt mit den alten Daten fehl.
  • PHPCS, PHPStan, check-i18n.py, build-release.sh und verify-package.py: alle grün.
  • E2E (admin-workflow.spec.js), neuer Test: Tab 5 zeigt das JSON-LD, einen REST-Link und das Thema im Klartext. Der Test schlägt ohne den Fix fehl.
  • Lokal liefen die übrigen Admin-Tests grün bis auf zwei. Beide scheitern an der Testumgebung, nicht am Code: Es fehlte die E2E-Saat, und der PHP-Testserver beantwortet keine Anfragen an sich selbst. Die CI fährt beides in wp-env.

Nicht in diesem PR

  • Das Raster aus dem Block FORMULAR-DESIGN (v2.36.0) greift nicht. Die Regeln hängen an .cf-container__tab. Carbon Fields 3.6.9 und 3.6.11 erzeugen diese Klasse aber nicht, das Panel heißt .cf-container__fields. Die Datumsfelder stehen deshalb nicht nebeneinander. Die Regeln umzuhängen, würde das ganze Formular neu anordnen, deshalb ist das eine eigene Änderung.
  • Englische Texte im Wiederholungsfeld: „There are no entries yet“ und „Add Distribution“ stammen aus der JavaScript-Oberfläche von Carbon Fields. Für sie liefert das Paket keine deutsche Übersetzung mit.

Generated by Claude Code

…EADME (v2.47.3)

- Tab 5 „Vorschau": Das HTML entstand bei der Feldregistrierung, bevor
  WordPress den Datensatz kennt; get_the_ID() lieferte false, JSON-LD und
  „REST-Endpoint öffnen" fehlten bei jedem Datensatz. set_html() bekommt
  jetzt einen Callback, den Carbon Fields erst beim Rendern aufruft.
- Admin-CSS: <fieldset> und Flex-Kinder mit min-width: 0, damit lange
  JSON-LD-Zeilen im Code-Block scrollen statt den Kasten aufzuspannen.
- Live-Vorschau: Thema als Beschriftung statt EU-URI, Dateigröße wie auf
  der Download-Card; Neuberechnung beim Reiterwechsel, weil die
  Mehrfachauswahl kein input/change-Ereignis auslöst.
- Beispieldaten: Zwei Datensätze trugen den HVD-Code c_b79e35eb als
  Schlagwort, jetzt „Mobilität". Test verbietet Vokabular-Codes als
  Schlagworte.
- README: Screenshots aus einem deutschen WordPress mit den
  Beispieldaten (docs/screenshots/, nicht im Plugin-ZIP).
- E2E: Tab 5 zeigt JSON-LD, REST-Link und das Thema im Klartext.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JB1xUQM892bVZ4Yv3MZjvq
@daimpad
daimpad merged commit b7372db into main Nov 6, 2026
22 checks passed
@daimpad
daimpad deleted the claude/fix-data-prep-errors-kJYpl branch November 6, 2026 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants