Skip to content

UI improvements: diff rendering + 20 Twig templates + asset bundle#103

Closed
sjelfull wants to merge 1 commit into
project-config-tracking-2from
ui-improvements-diff-rendering-2
Closed

UI improvements: diff rendering + 20 Twig templates + asset bundle#103
sjelfull wants to merge 1 commit into
project-config-tracking-2from
ui-improvements-diff-rendering-2

Conversation

@sjelfull
Copy link
Copy Markdown
Owner

  • Add DiffRenderer service wrapping jfcherng/php-diff + caxy/php-htmldiff
  • Register diffRenderer component + expose via craft.audit.diff in Twig
  • AuditDiffAsset bundle + diff.css (namespaced, dark-mode via CSS custom
    properties and both media-query + body.dark selectors)
  • 20 per-handler Twig templates under src/templates/_diff/:
    index (dispatcher), plain, color, richtext, date, boolean, option,
    multi-option, relation, money, table, matrix, neo, supertable, vizy,
    seo, generic, config, error, truncated
  • Register asset bundle on audit/index and audit/details actions
  • 14 DiffRendererTest cases (truncation, identical, empty, HTML/text/JSON,
    XSS escape, malformed HTML fallback)
  • Tests: 114 passed, 1 skipped (was 100 passed) — +14

- Add DiffRenderer service wrapping jfcherng/php-diff + caxy/php-htmldiff
- Register diffRenderer component + expose via craft.audit.diff in Twig
- AuditDiffAsset bundle + diff.css (namespaced, dark-mode via CSS custom
  properties and both media-query + body.dark selectors)
- 20 per-handler Twig templates under src/templates/_diff/:
  index (dispatcher), plain, color, richtext, date, boolean, option,
  multi-option, relation, money, table, matrix, neo, supertable, vizy,
  seo, generic, config, error, truncated
- Register asset bundle on audit/index and audit/details actions
- 14 DiffRendererTest cases (truncation, identical, empty, HTML/text/JSON,
  XSS escape, malformed HTML fallback)
- Tests: 114 passed, 1 skipped (was 100 passed) — +14
@sjelfull
Copy link
Copy Markdown
Owner Author

Closing — duplicate created by gs stack submit when its tracking metadata fell out of sync. The original PR has been force-updated with the corrected SHA. See #93.

@sjelfull sjelfull closed this Apr 27, 2026
@sjelfull sjelfull deleted the ui-improvements-diff-rendering-2 branch April 27, 2026 20:59
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.

1 participant