Skip to content

Conversation

@SoonIter
Copy link
Member

Summary

fix(theme): th width should be same with td

Related Issue

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copilot AI review requested due to automatic review settings November 24, 2025 03:49
@netlify
Copy link

netlify bot commented Nov 24, 2025

Deploy Preview for rspress-v2 ready!

Name Link
🔨 Latest commit 962d4f4
🔍 Latest deploy log https://app.netlify.com/projects/rspress-v2/deploys/6923d5e60d89fb00085eca67
😎 Deploy Preview https://deploy-preview-2804--rspress-v2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a table styling inconsistency by aligning the horizontal padding of table header cells (th) with table data cells (td). The horizontal padding is changed from 2rem to 1.5rem to match the existing td padding.

  • Changed th horizontal padding from 2rem to 1.5rem to match td cells

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Contributor

Rsdoctor Bundle Diff Analysis

📁 web

Path: website/doc_build/web/rsdoctor-data.json

📌 Baseline Commit: ce66301614 | PR: #2800

Metric Current Baseline Change
📊 Total Size 14.4 MB 14.4 MB 2.0 B (0.0%)
📄 JavaScript 13.8 MB 13.8 MB 0 B (0.0%)
🎨 CSS 109.9 KB 109.9 KB 2.0 B (0.0%)
🌐 HTML 0 B 0 B N/A
📁 Other Assets 496.9 KB 496.9 KB 0 B (0.0%)

📦 Download Diff Report: web Bundle Diff

Generated by Rsdoctor GitHub Action

@SoonIter SoonIter merged commit 84e750d into main Nov 24, 2025
16 checks passed
@SoonIter SoonIter deleted the syt/fix-table-padding branch November 24, 2025 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants