Skip to content

Conversation

@ghanshyam2005singh
Copy link
Contributor

@ghanshyam2005singh ghanshyam2005singh commented Dec 9, 2025

This PR addresses the UI bug where the "Check It Out" button in the "Create circuits" section and "Send" button in contact page changed from green to red on hover or active state. The button now maintains a green color scheme consistent with CircuitVerse branding.


Problem

  • The "Check It Out" and "send" button was transitioning from green to red on hover/active, which is inconsistent with the expected color scheme.

Solution

  • Added SCSS overrides in assets/scss/_mixins.scss:
    • Default color: #42b983 (CircuitVerse green)
    • Hover/Active color: #368f6e (darker green)
  • No submodule files were modified; all changes are local and safe from future submodule updates.
  • Ensures the button remains green on hover/active, matching the main site’s branding.

Screenshot

Screencast.from.2025-12-09.17-02-35.webm

Closes #255

Summary by CodeRabbit

  • Style
    • Updated primary button styling: default green remains, and hover/focus/active/pressed states now use a deeper green (#368f6e) across primary button variants to ensure consistent interaction visuals.

✏️ Tip: You can customize this high-level summary in your review settings.

Signed-off-by: Ghanshyam Singh <ghanshyam2005singh@gmail.com>
@coderabbitai
Copy link

coderabbitai bot commented Dec 9, 2025

Walkthrough

Adds a CSS override block that sets a darker green background and border color (#368f6e) for interactive states (hover, active, focus, pressed) of .btn-primary and .container .btn-primary, complementing existing button theme rules that use #40ba84.

Changes

Cohort / File(s) Summary
Button styling override
assets/scss/_mixins.scss
Appends a CSS override block targeting .btn-primary and .container .btn-primary pseudo-classes (hover, active, focus, pressed), applying a darker green (#368f6e) for background and border to modify runtime interactive states.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

  • Check hex values (#368f6e and existing #40ba84) for design consistency.
  • Confirm selector specificity covers the affected buttons and does not unintentionally override other theme rules.

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: fixing button hover colors to stay green instead of turning red, which aligns with the CSS override changes made to _mixins.scss.
Linked Issues check ✅ Passed The PR successfully addresses issue #255 by adding SCSS overrides that enforce CircuitVerse green colors (#368f6e for hover/active states) across primary button variants, preventing the red color transition.
Out of Scope Changes check ✅ Passed The changes are focused solely on fixing the button hover color issue in _mixins.scss and do not introduce unrelated modifications outside the scope of issue #255.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8650e67 and 5df3085.

📒 Files selected for processing (1)
  • assets/scss/_mixins.scss (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • assets/scss/_mixins.scss

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.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8fd3c41 and 8650e67.

📒 Files selected for processing (1)
  • assets/scss/_mixins.scss (1 hunks)

@tachyons tachyons merged commit 24cfa62 into CircuitVerse:master Dec 9, 2025
7 checks passed
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.

UI Bug: Button color transition failure on hover (Green to Red)

2 participants