fix(docs-site): restore scrollable tables (layout overflow under right TOC) - #1003
Merged
Conversation
…id wide tables under the TOC
The global 'table { display: table; width: 100%; }' in custom.css
disabled Infima's display:block + overflow-x:auto markdown-table
behavior, so any table wider than the content column (e.g. the
telemetry field table) overflowed beneath the right-hand TOC and
crushed its last column into extreme row heights. Wide tables now
scroll inside the column again.
Deploying mcpproxy-docs with
|
| Latest commit: |
a4d805f
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://e8fe0ba4.mcpproxy-docs.pages.dev |
| Branch Preview URL: | https://docs-table-overflow-fix.mcpproxy-docs.pages.dev |
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
📦 Build ArtifactsWorkflow Run: View Run Available Artifacts
How to DownloadOption 1: GitHub Web UI (easiest)
Option 2: GitHub CLI gh run download 32020787775 --repo smart-mcp-proxy/mcpproxy-go
|
Dumbris
enabled auto-merge (squash)
August 17, 2026 10:55
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Dumbris
added a commit
that referenced
this pull request
Aug 17, 2026
…) (#1004) Follow-up to #1003, which was based on a wrong assumption: this Infima version ships `table { display: table }` (its `overflow: auto` is inert on that display type — verified from live computed styles), so wide tables still slid under the right TOC after #1003 deployed. Real fix, scoped to `.markdown table`: - `display: block; width: fit-content; max-width: 100%; overflow-x: auto` — tables become genuine scroll containers clipped to the content column. - `.markdown table code { white-space: pre-wrap; word-break: break-word }` — long code tokens (JSON examples, hashes) wrap inside cells instead of forcing one column to full unbroken width and starving the rest (this was the 156px-row-height cause). **Verified on a local build** with browser computed styles + screenshots: telemetry table now fits the column exactly (643/643px, was 1364px overflowing), all three columns visible, rows 156→79px, TOC clear.
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
2 participants
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.
The custom.css global
table { display: table; width: 100%; }override disabled Docusaurus/Infima's defaultdisplay: block; overflow-x: automarkdown-table behavior. Result (visible on /features/telemetry/): wide tables escaped the content column, slid under the right-hand TOC, and the squeezed trailing column produced extreme row heights.Fix: drop the
displayoverride (framework default restored — wide tables scroll inside the column); keepwidth: 100%so the scroll container spans the column. Site-wide effect is a return to stock Docusaurus table behavior. Docusaurus build clean (165 docs).