Skip to content

docs: remove table styling and use default values#9998

Merged
mendonk merged 2 commits into
mainfrom
docs-tables-css-fix
Sep 26, 2025
Merged

docs: remove table styling and use default values#9998
mendonk merged 2 commits into
mainfrom
docs-tables-css-fix

Conversation

@mendonk
Copy link
Copy Markdown
Collaborator

@mendonk mendonk commented Sep 26, 2025

Revert table styling and let Infima and Docusaurus handle it.

Summary by CodeRabbit

  • Documentation
    • Updated docs appearance: removed custom markdown table styles. Tables now render with the default theme for a more consistent look across pages.
  • Style
    • Simplified CSS by eliminating table-specific rules, reducing overrides and potential visual inconsistencies.

@mendonk mendonk requested a review from aimurphy September 26, 2025 14:08
@mendonk mendonk self-assigned this Sep 26, 2025
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Sep 26, 2025

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

Removed all CSS rules targeting markdown tables in docs/css/custom.css, including container, header, body, cell, row, and code-in-table styles. No new CSS was added.

Changes

Cohort / File(s) Change Summary
Docs CSS cleanup
docs/css/custom.css
Deleted entire markdown table styling block: .markdown table, thead th/tr, tbody td/tr, fixed-width th/td, last-child border rules, and table code styles.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

documentation

Suggested reviewers

  • kurtenj

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Test File Naming And Structure ❓ Inconclusive Only docs/css/custom.css was modified in this pull request, so there are no new or updated test files to examine against the required naming and structural guidelines; without any test changes, I cannot determine whether the criteria are met. Please add or reference the relevant tests so their naming and structure can be reviewed, or confirm that no test updates are needed for this change.
✅ Passed checks (6 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title accurately summarizes the primary change of removing custom table styling in the docs and defaulting to upstream styles; it is concise, clear, and matches the changeset.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
Test Coverage For New Implementations ✅ Passed The PR only removes custom markdown table styling from docs/css/custom.css and does not introduce new functionality, bug fixes, or components; therefore no new or updated test cases are required under the check criteria, and none are included.
Test Quality And Coverage ✅ Passed The pull request only removes custom table styling from the documentation CSS and does not introduce or modify executable code paths, asynchronous functions, or API endpoints. Because no functional behavior changes are being implemented, no automated tests are expected or necessary for this update. Consequently, the absence of added or updated tests is acceptable for this documentation-only change.
Excessive Mock Usage Warning ✅ Passed No test files are modified in this pull request; the only change removes custom table styling from docs/css/custom.css, so there is no opportunity for excessive mock usage to arise or to evaluate. Therefore, the Excessive Mock Usage Warning check is not applicable and the criteria are satisfied by default.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Sep 26, 2025
@github-actions

This comment has been minimized.

@github-actions github-actions Bot added documentation Improvements or additions to documentation and removed documentation Improvements or additions to documentation labels Sep 26, 2025
@sonarqubecloud
Copy link
Copy Markdown

@github-actions github-actions Bot added documentation Improvements or additions to documentation and removed documentation Improvements or additions to documentation labels Sep 26, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 26, 2025

Build successful! ✅
Deploying docs draft.
Deploy successful! View draft

@github-actions github-actions Bot added the lgtm This PR has been approved by a maintainer label Sep 26, 2025
@mendonk mendonk added this pull request to the merge queue Sep 26, 2025
Merged via the queue into main with commit b6a4385 Sep 26, 2025
17 checks passed
@mendonk mendonk deleted the docs-tables-css-fix branch September 26, 2025 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation lgtm This PR has been approved by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants