Repository navigation
Screenshots im README, JSON-LD-Vorschau repariert (v2.47.3) - #152
Merged
Merged
Conversation
…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
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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)formular.png,katalog.pngdatensatzliste.pngvorschau.pngbatch-import.pngqualitaet.pngkatalog-detail.pngkatalog-filter.pngAlle Aufnahmen sind in doppelter Auflösung.
verify-package.pybestätigt, dassdocs/nicht im ZIP landet.Fehlerbehebungen
get_preview_html()lief schon beicarbon_fields_register_fields. Zu diesem Zeitpunkt liefertget_the_ID()nochfalse. 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.<fieldset>von Carbon Fields hat ab Werkmin-width: min-content. Gesetzt ist jetztmin-width: 0auffieldset,.cf-container__fieldsund die Felder darin. Bei 1280 px laufen alle fünf Reiter ohne horizontales Scrollen.input/changeauslöst.c_b79e35ebals Schlagwort. Jetzt steht dort „Mobilität“, in CSV und JSON.Tests
check-i18n.py,build-release.shundverify-package.py: alle grün.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.Nicht in diesem PR
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.Generated by Claude Code