Skip to content

Preserve Markdown table formatting in summary text view#120

Open
sayedrisat wants to merge 1 commit into
HexmosTech:mainfrom
sayedrisat:fix/render-summary-tables
Open

Preserve Markdown table formatting in summary text view#120
sayedrisat wants to merge 1 commit into
HexmosTech:mainfrom
sayedrisat:fix/render-summary-tables

Conversation

@sayedrisat

@sayedrisat sayedrisat commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Preserves Markdown table structure in the review summary Text view.

Why

The slideshow summary sanitizer already allows table elements, but the regular summary Text view sanitizer stripped table tags. That flattened Markdown tables into hard-to-scan text even though the same summary could render as a table elsewhere.

This keeps the sanitizer behavior consistent and adds compact dark-theme table styling for the Text view.

Before / After

Before and after summary table rendering

Before: table content was flattened into plain text.

After: headers, rows, borders, and dark-theme styling are preserved.

Validation

  • node --test internal/staticserve/static/components/*.test.mjs
  • git diff --check

Note: make test-js could not run locally because make is not installed in this PowerShell environment; the equivalent Node test command from the Makefile passed.

@LinceMathew

Copy link
Copy Markdown
Contributor

Hi @sayedrisat,
Did you encounter this issue in a real review triggered from git-lrc? The summary for a review is configured to return as a bullet list rather than a table. See reference.
For example:
image

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