Skip to content

refactor: replace hard-coded theme colors - #655

Merged
jbampton merged 2 commits into
NextCommunity:mainfrom
rohit-singh121:fix/theme-color-variables
Oct 10, 2026
Merged

jbampton merged 2 commits into
NextCommunity:mainfrom
rohit-singh121:fix/theme-color-variables

Conversation

@rohit-singh121

Copy link
Copy Markdown
Contributor

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 •

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 2f0945a...6dcd87e 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 10, 2026 11:49p.m. Review ↗
Secrets Oct 10, 2026 11:49p.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 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
@jbampton jbampton self-assigned this Oct 10, 2026
@jbampton jbampton added this to the Hacktoberfest 2026 milestone Oct 10, 2026
@jbampton jbampton added css good first issue Good for newcomers labels Oct 10, 2026

@jbampton jbampton left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks 🏂

@jbampton
jbampton merged commit 331c489 into NextCommunity:main Oct 10, 2026
16 checks passed
@jbampton

Copy link
Copy Markdown
Member

Hey @rohit-singh121 you will need a GitHub sponsors profile to receive payment

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

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Replace hard-coded colors with reusable theme variables

4 participants