Skip to content

fix table heading rendering - #1138

Merged
adamdotdevin merged 2 commits into
anomalyco:devfrom
CodinCat:fix-table-heading-rendering
Jul 19, 2025
Merged

adamdotdevin merged 2 commits into
anomalyco:devfrom
CodinCat:fix-table-heading-rendering

Conversation

@CodinCat

@CodinCat CodinCat commented Jul 19, 2025 •

Copy link
Copy Markdown
Contributor

fix #688

These \n have existed since the first commit. I'm not sure what's the purpose of it, but they are breaking the table rendering. The entire table looks quite weird and the headings are missing.

The background color of the table also doesn't look right, but it seems to be an issue of glamour. They've already fixed it but just haven't released it yet.

Before

before

After

after

a small issue is that the color of ColumnSeparator is different from CenterSeparator and RowSeparator (these two seem to be TextMuted). not sure where they inherit the colors from. at least the table looks more normal now


Update:

just found that if we keep the BlockSuffix the border colors will look consistent. I've updated the code. it adds some extra spaces but I assume it looks better?

BlockSuffix

@adamdotdevin
adamdotdevin merged commit 1676f8b into anomalyco:dev Jul 19, 2025
@CodinCat

Copy link
Copy Markdown
Contributor Author

oh wow you're merging so fast not sure if you saw the update 😂

@adamdotdevin

Copy link
Copy Markdown
Member

oh wow you're merging so fast not sure if you saw the update 😂

i saw it!

AIALRA-0 pushed a commit to AIALRA-0/opencode-turn-engine that referenced this pull request Jun 10, 2026
meSingh pushed a commit to meSingh/opencode that referenced this pull request Sep 6, 2026
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.

AI response markdown rendering tables

2 participants