Repository navigation
style(markdown): visible gridlines on rendered tables - #1166
Open
probertson wants to merge 1 commit into
Open
probertson wants to merge 1 commit into
probertson wants to merge 1 commit into
Conversation
The ⌘M chat view had no table styling (cells ran together with no lines), and the output view and editor preview drew their lines with the 10% separator token. One rule group now covers both containers: 22% gridlines, a tinted header row, horizontal scroll inside a table wider than the view, and unaligned headers start-aligned to match their cells (a column's Markdown alignment, an `align` attribute, still wins). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Rendered Markdown tables now have visible gridlines in every reading view.
Before: the ⌘M chat view had no table styling at all, so cells ran together with no lines. The ⌘M output view and the editor's Markdown preview drew lines with
--border, a 10% separator that made them hard to see.Now (one rule group for both containers,
.term-md__contentand.term-chat__text):---:,:---:) still applies. It arrives as analignattribute, which an authortext-alignwould override, so only headers without one are start-aligned (to match their cells instead of the browser's centred default).The chat view's plan and question cards render through the same container, so their tables get the same treatment. Sticky notes are unchanged.
How it was checked
styles.markdown-table.test.tspins the rules for both containers and that cells set notext-align. Thesrc/renderer/stylessuites pass.display: blockwith overflow, and the right-aligned column kept its alignment. Checked a screenshot before and after the header-alignment tweak.Not checked: the light theme and the output view/editor preview by hand (same rules, covered by the test).
🤖 Generated with Claude Code