Skip to content

Conversation

@SoonIter
Copy link
Member

@SoonIter SoonIter commented Nov 20, 2025

Summary

fix(theme): css vars color

Related Issue

introduced in #2771

Checklist

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

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

netlify bot commented Nov 20, 2025

Deploy Preview for rspress-v2 ready!

Name Link
🔨 Latest commit bae7b55
🔍 Latest deploy log https://app.netlify.com/projects/rspress-v2/deploys/691e85cb2d5d58000824993a
😎 Deploy Preview https://deploy-preview-2791--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 dark mode CSS by extracting a hardcoded color value into a CSS variable for better maintainability and consistency with the light mode implementation.

  • Introduces a CSS variable --rp-home-hero-secondary-color in dark mode to match the light mode pattern
  • Replaces hardcoded #a673ff with the new CSS variable reference in the gradient definition

💡 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

Metric Current Baseline Change
📊 Total Size 14.3 MB 14.3 MB -8.3 KB (-0.1%)
📄 JavaScript 13.7 MB 13.8 MB -8.3 KB (-0.1%)
🎨 CSS 109.6 KB 109.5 KB 66.0 B (0.1%)
🌐 HTML 0 B 0 B N/A
📁 Other Assets 497.3 KB 497.3 KB 0 B (0.0%)

📦 Download Diff Report: web Bundle Diff

Generated by Rsdoctor GitHub Action

@SoonIter SoonIter merged commit be52ec0 into main Nov 20, 2025
16 checks passed
@SoonIter SoonIter deleted the syt/fix-theme-var branch November 20, 2025 06:27
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