fix(css): change table cell vertical alignment from middle to top#40
Draft
fix(css): change table cell vertical alignment from middle to top#40
Conversation
- Tables now use top vertical alignment for better readability - Applies to all table cells (td and th) in markdown content - Fixes issue where varying cell content heights made tables hard to scan Fixes #33
Contributor
Author
Code Review SummaryWhat This PR DoesThis PR improves table readability by changing the vertical alignment of table cells from middle to top alignment:
How It Solves the IssueBy setting
Potential Issues & ConsiderationsStrengths:
Areas to Watch:
Code Quality:
Testing:
Recommendation: ✅ Approve - Standard UX improvement with minimal risk. Top alignment is the conventional and more readable choice for documentation tables. |
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.
Summary
Fixes table cell vertical alignment to improve readability and scannability of documentation tables.
Problem
Tables in the documentation were using middle (center) vertical alignment, which made content appear awkwardly positioned when row heights varied. This was particularly problematic for tables with different amounts of text in cells.
Solution
Changed all table cells (both
tdandth) to usevertical-align: topin markdown content.Changes
src/css/custom.cssfor.theme-doc-markdown tablecellsTesting
Visual Impact
Tables will now have content aligned to the top of cells, making them:
Screenshots
See issue #33 for before/after comparison.
Fixes #33