Skip to content

fix(theme): nprogress bar color#2770

Merged
Timeless0911 merged 2 commits intomainfrom
syt/fix-nprogress
Nov 18, 2025
Merged

fix(theme): nprogress bar color#2770
Timeless0911 merged 2 commits intomainfrom
syt/fix-nprogress

Conversation

@SoonIter
Copy link
Member

Summary

fix(theme): nprogress bar color

Related Issue

Checklist

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

Copilot AI review requested due to automatic review settings November 17, 2025 11:30
@netlify
Copy link

netlify bot commented Nov 17, 2025

Deploy Preview for rspress-v2 ready!

Name Link
🔨 Latest commit cef72c7
🔍 Latest deploy log https://app.netlify.com/projects/rspress-v2/deploys/691b074ff58b7b0009959ea6
😎 Deploy Preview https://deploy-preview-2770--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 the nprogress bar color by reordering CSS imports and refactoring style organization. The fix ensures that the nprogress library's default styles load first, allowing the custom brand color override to work correctly.

  • Moved nprogress CSS import to load before custom styles, ensuring proper CSS cascade
  • Increased CSS specificity for the color override using :root #nprogress .bar
  • Refactored component-specific styles into a dedicated NavTitle.scss file for better organization

Reviewed Changes

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

Show a summary per file
File Description
packages/core/src/theme/styles/index.ts Moved nprogress CSS import to the top to ensure library defaults load before custom overrides
packages/core/src/theme/styles/base.css Updated nprogress bar selector with higher specificity and removed redundant positioning styles now provided by the library
packages/core/src/theme/components/Nav/index.scss Removed logo and dark mode styles (moved to NavTitle.scss)
packages/core/src/theme/components/Nav/NavTitle.tsx Added import for the new NavTitle.scss stylesheet
packages/core/src/theme/components/Nav/NavTitle.scss Created new file containing logo and dark mode styles for better component colocation

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

@Timeless0911 Timeless0911 merged commit 4059e7b into main Nov 18, 2025
15 of 16 checks passed
@Timeless0911 Timeless0911 deleted the syt/fix-nprogress branch November 18, 2025 05:55
@github-actions
Copy link
Contributor

Rsdoctor Bundle Diff Analysis

📁 web

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

⚠️ No baseline data found - Unable to perform comparison analysis

Metric Current Baseline Change
📊 Total Size 14.3 MB - -
📄 JavaScript 13.7 MB - -
🎨 CSS 109.0 KB - -
🌐 HTML 0 B - -
📁 Other Assets 496.7 KB - -

Generated by Rsdoctor GitHub Action

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