Skip to content

refactor: replace hard-coded theme colors - #655

Open
rohit-singh121 wants to merge 1 commit into
NextCommunity:mainfrom
rohit-singh121:fix/theme-color-variables
Open

refactor: replace hard-coded theme colors#655
rohit-singh121 wants to merge 1 commit into
NextCommunity:mainfrom
rohit-singh121:fix/theme-color-variables

Conversation

@rohit-singh121

@rohit-singh121 rohit-singh121 commented Oct 8, 2026

Copy link
Copy Markdown

Description

Replaced appropriate hard-coded theme colors with reusable CSS theme variables.

Changes

  • Added reusable primary, success, info, and light text color variables.
  • Replaced repeated hard-coded color values with CSS variables.
  • Added RGB variables for colors used with rgba().
  • Preserved special-purpose colors and existing visual behavior.

Related Issue

Closes #643

@deepsource-io

deepsource-io Bot commented Oct 8, 2026 •
edited
Loading

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 0eed0f7...1e5eb88 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
JavaScript Oct 8, 2026 1:16p.m. Review ↗
Secrets Oct 8, 2026 1:16p.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@jbampton
jbampton marked this pull request as draft October 8, 2026 23:31
@jbampton
jbampton requested a balanced review from Copilot October 10, 2026 06:58
Copilot AI reviewed Oct 10, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The substitutions preserve existing color values and CSS behavior without unresolved issues.

0 open findings

What changed in this PR

Centralizes repeated theme colors in CSS variables while preserving existing visuals.

Changes:

  • Adds primary, success, info, light-text, and RGB variables.
  • Replaces matching hard-coded colors across buttons, level styling, and console UI.
File Description
src/​assets/​css/​style.css Defines and applies reusable theme color variables.

🧠 Review effort: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@rohit-singh121
rohit-singh121 marked this pull request as ready for review October 10, 2026 11:59

This branch has not been deployed

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

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@BaseMax BaseMax Awaiting requested review from BaseMax BaseMax is a code owner
@jbampton jbampton Awaiting requested review from jbampton jbampton is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

Next
Status: No status

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Replace hard-coded colors with reusable theme variables

3 participants

@rohit-singh121 @Sidhant2104