Skip to content

bgcolour fix for dark theme fix of #349#365

Open
priyanshubarai wants to merge 1 commit intozhravan:mainfrom
priyanshubarai:dark-theme-fix
Open

bgcolour fix for dark theme fix of #349#365
priyanshubarai wants to merge 1 commit intozhravan:mainfrom
priyanshubarai:dark-theme-fix

Conversation

@priyanshubarai
Copy link

@priyanshubarai priyanshubarai commented Feb 14, 2026

Before
before

After
after

the Quote Card in Dark theme seems more consistent with the rest of Github dark UI

Summary by CodeRabbit

Release Notes

  • Style
    • Updated the dark theme background color for improved visual appearance and consistency.

@vercel
Copy link

vercel bot commented Feb 14, 2026

@priyanshubarai is attempting to deploy a commit to the shravan20's projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link

coderabbitai bot commented Feb 14, 2026

Walkthrough

Two theme configuration files were updated to adjust the dark theme background color from #151515 to #161B22, maintaining visual consistency across the codebase. No functional or structural changes were introduced.

Changes

Cohort / File(s) Summary
Dark Theme Background Update
frontend/src/util/themes/index.js, src/themes/themes.js
Updated dark theme background color from #151515 to #161B22 across theme definitions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A darker shade now graces the night,
#161B22 brings balance and light,
Two files aligned in harmonious hue,
The theme refresh—simple and true!

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'bgcolour fix for dark theme fix of #349' directly describes the main change—updating the background color for the dark theme—and references the related issue.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant