fix(docs): avoid duplicate background warning#1214
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughA CSS feature query is added to conditionally apply ChangesPreview Sidebar Color-Mix Fallback
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Deploying quickadd with
|
| Latest commit: |
3ecc603
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://5ad051ae.quickadd.pages.dev |
| Branch Preview URL: | https://fix-docs-background-warning.quickadd.pages.dev |
Avoid the duplicate
backgroundwarning in the docs homepage CSS.The preview sidebar used a plain
backgroundfallback followed by acolor-mix()enhancement in the same rule, which triggered an unexpected duplicate property warning. This keeps the fallback as the base declaration and moves the enhanced color into an@supportsblock so unsupported browsers still get the fallback while supported browsers apply the mixed background.Verification:
bun run buildfromdocs/obsidian vault=dev eval, confirmingCSS.supports(...)is true, the mixed computed background applies, anddev:errorsreports no errors.Summary by CodeRabbit