Skip to content

chore: remove unused min-height style#2917

Merged
Timeless0911 merged 2 commits intomainfrom
syt/remove-unused-code
Dec 22, 2025
Merged

chore: remove unused min-height style#2917
Timeless0911 merged 2 commits intomainfrom
syt/remove-unused-code

Conversation

@SoonIter
Copy link
Copy Markdown
Member

Summary

chore: remove unused min-height style

Related Issue

close #2909

Checklist

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

Copilot AI review requested due to automatic review settings December 22, 2025 03:28
@netlify
Copy link
Copy Markdown

netlify bot commented Dec 22, 2025

Deploy Preview for rspress-v2 ready!

Name Link
🔨 Latest commit 52b5f7f
🔍 Latest deploy log https://app.netlify.com/projects/rspress-v2/deploys/6948bb6166806a0008e86df2
😎 Deploy Preview https://deploy-preview-2917--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.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 22, 2025

Rsdoctor Bundle Diff Analysis

Found 3 projects in monorepo, 1 project with changes.

📊 Quick Summary
Project Total Size Change
node 9.1 MB 0
node_md 1.1 MB 0
web 15.0 MB -61.0 B (-0.0%)
📋 Detailed Reports (Click to expand)

📁 web

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

📌 Baseline Commit: 3ec6aecf50 | PR: #2911

Metric Current Baseline Change
📊 Total Size 15.0 MB 15.0 MB -61.0 B (-0.0%)
📄 JavaScript 14.3 MB 14.3 MB -29.0 B (-0.0%)
🎨 CSS 115.4 KB 115.4 KB -32.0 B (-0.0%)
🌐 HTML 0 B 0 B 0
📁 Other Assets 516.6 KB 516.6 KB 0

📦 Download Diff Report: web Bundle Diff

Generated by Rsdoctor GitHub Action

Copy link
Copy Markdown
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 removes unused code from the DocLayout component, specifically eliminating the min-height: 60vh style declarations and an unused defaultGroupTitle parameter from the Overview component.

  • Removed min-height: 60vh CSS declarations from two layout containers
  • Removed unused defaultGroupTitle parameter and its associated destructured variable from the Overview component

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
packages/core/src/theme/layout/DocLayout/index.scss Removed unused min-height: 60vh style from &__overview and &__doc-container classes
packages/core/src/theme/components/Overview/index.tsx Removed unused defaultGroupTitle prop from component interface and its destructured assignment

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

@Timeless0911 Timeless0911 merged commit c83c6d8 into main Dec 22, 2025
9 checks passed
@Timeless0911 Timeless0911 deleted the syt/remove-unused-code branch December 22, 2025 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Some content doesn't render when exporting as PDF unexpectedly after v2.0.0-rc.1

3 participants