Conversation
There was a problem hiding this comment.
🟡 Changes recommended
The new display: table combined with overflow: auto can break horizontal scrolling for wide tables due to inconsistent overflow behavior on table display types.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
This PR adjusts the GitHub theme’s table styling to change how tables size and constrain their width in preview mode, and regenerates the shipped bundles accordingly.
Changes:
- Update
.markdown-body tableto usedisplay: table,width: auto, and a cappedmax-width. - Regenerate the full and lite distribution bundles to include the updated CSS.
File summaries
| File | Description |
|---|---|
| styles/themes/github/base.css | Changes the core GitHub theme table layout rules used by the preview. |
| dist/markedit-preview.js | Regenerated full bundle reflecting the theme CSS change. |
| dist/lite/markedit-preview.js | Regenerated lite bundle reflecting the theme CSS change. |
Review details
- Files reviewed: 1/3 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
There was a problem hiding this comment.
🟡 Changes recommended
The new CSS uses fit-content and min() without fallbacks, which can regress behavior on hosts that don’t support those constructs unless compatibility fallbacks are added and bundles are regenerated.
Get a fresh assessment by requesting another Copilot review.
Review details
- Files reviewed: 1/3 changed files
- Comments generated: 1
- Review effort level: Lite
There was a problem hiding this comment.
🟢 Approval recommended
The change is small, consistent with existing CSS feature usage in this theme, and the generated dist artifacts were updated accordingly.
Review details
- Files reviewed: 1/3 changed files
- Comments generated: 0 new
- Review effort level: Lite
No description provided.