Skip to content

fix(theme/callout): link in callout should use currentColor#2838

Merged
Timeless0911 merged 2 commits intomainfrom
syt/fix-link-border
Dec 3, 2025
Merged

fix(theme/callout): link in callout should use currentColor#2838
Timeless0911 merged 2 commits intomainfrom
syt/fix-link-border

Conversation

@SoonIter
Copy link
Copy Markdown
Member

@SoonIter SoonIter commented Dec 3, 2025

Summary

fix(theme/callout): link in callout should use currentColor

image

Related Issue

Checklist

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

Copilot AI review requested due to automatic review settings December 3, 2025 10:00
@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 3, 2025

Deploy Preview for rspress-v2 ready!

Name Link
🔨 Latest commit dea2e9a
🔍 Latest deploy log https://app.netlify.com/projects/rspress-v2/deploys/69300b797a353d0008f5c702
😎 Deploy Preview https://deploy-preview-2838--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
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 fixes link styling in callouts to use currentColor instead of the brand color, ensuring links maintain visual consistency with their callout's theme color.

  • Removed redundant link styles from callout component that were already defined in the global doc styles
  • Updated hover border color to use currentColor instead of brand color for contextual theming

Reviewed changes

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

File Description
packages/core/src/theme/components/Callout/index.scss Removed duplicate link styling (font-weight, transition, border-bottom) from callout content as these are already handled by the global .rp-link styles
packages/core/src/theme/components/DocContent/doc.scss Changed link hover border from brand color to currentColor to respect the current text color context (e.g., callout theme colors)

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 3, 2025

Rsdoctor Bundle Diff Analysis

📁 web

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

📌 Baseline Commit: 197d9e06a8 | PR: #2834

Metric Current Baseline Change
📊 Total Size 14.5 MB 14.5 MB -56.0 B (-0.0%)
📄 JavaScript 13.9 MB 13.9 MB 0 B (0.0%)
🎨 CSS 113.7 KB 113.8 KB -56.0 B (-0.0%)
🌐 HTML 0 B 0 B N/A
📁 Other Assets 499.9 KB 499.9 KB 0 B (0.0%)

📦 Download Diff Report: web Bundle Diff

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