Skip to content

Conversation

@SoonIter
Copy link
Member

@SoonIter SoonIter commented Sep 16, 2025

Summary

feat(theme/style)!: keep the html style be consistent with markdown syntax and .rp-not-doc escape hatch (#2579)

Related Issue

#2569

chore(theme/style): keep the html syntax in markdown original style
image
before
image
after
image

Checklist

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

Copilot AI review requested due to automatic review settings September 16, 2025 04:18
@netlify
Copy link

netlify bot commented Sep 16, 2025

Deploy Preview for rspress-v2 ready!

Name Link
🔨 Latest commit d92f403
🔍 Latest deploy log https://app.netlify.com/projects/rspress-v2/deploys/68c92044821e76000867c1aa
😎 Deploy Preview https://deploy-preview-2600--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 refactors the theming system to unify HTML element styling between markdown syntax and HTML tags, introducing a new .rp-not-doc escape hatch for opting out of documentation styling.

  • Consolidates styling by moving MDX component styles to a centralized SCSS file
  • Introduces .rp-doc and .rp-not-doc CSS classes for controlling documentation styling
  • Simplifies MDX component implementations by removing inline Tailwind classes

Reviewed Changes

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

Show a summary per file
File Description
packages/theme-default/src/styles/doc.scss New centralized SCSS file containing all documentation element styles
packages/theme-default/src/layout/DocLayout/docComponents/*.tsx Simplified component implementations removing Tailwind classes
website/docs/*/guide/basic/use-mdx/_escape-hatch.tsx Updated example demonstrating new styling approach
packages/plugin-/static/global-components/.tsx Added rp-not-doc class to plugin components
e2e/fixtures/theme-css/doc/prose/index.mdx New test file demonstrating styling consistency

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@SoonIter SoonIter merged commit d1ca1d8 into main Sep 16, 2025
8 checks passed
@SoonIter SoonIter deleted the syt/keep-html-style-to-be-consistent-with-markdown-syntax branch September 16, 2025 08:51
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