Skip to content

Conversation

@SoonIter
Copy link
Member

Summary

fix: OverviewGroup and title margin

Related Issue

Checklist

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

Copilot AI review requested due to automatic review settings October 29, 2025 11:13
@SoonIter SoonIter changed the title fix: OverviewGroup and title margin fix(theme): OverviewGroup and title margin Oct 29, 2025
@netlify
Copy link

netlify bot commented Oct 29, 2025

Deploy Preview for rspress-v2 ready!

Name Link
🔨 Latest commit a19f191
🔍 Latest deploy log https://app.netlify.com/projects/rspress-v2/deploys/6902e59cc42db9000899816b
😎 Deploy Preview https://deploy-preview-2699--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 adds styling improvements for the documentation layout and overview components, including h5/h6 heading styles, word-breaking behavior, and a refactored conditional for the outline display.

  • Added styles for h5 and h6 headings in documentation content
  • Applied word-break: break-all to overview group titles and items to prevent overflow
  • Refactored the outline display conditional logic in DocLayout

Reviewed Changes

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

File Description
packages/core/src/theme/layout/DocLayout/index.tsx Refactored conditional rendering logic for the outline sidebar
packages/core/src/theme/components/Overview/OverviewGroup.scss Added word-break styling to prevent text overflow in titles and content items
packages/core/src/theme/components/DocContent/doc.scss Added h5 and h6 heading styles for consistent typography

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

Timeless0911
Timeless0911 previously approved these changes Oct 30, 2025
@SoonIter SoonIter merged commit 8a16abb into main Oct 30, 2025
8 checks passed
@SoonIter SoonIter deleted the syt/fix-overview branch October 30, 2025 06:48
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