Skip to content

Fix: contact form focus border color#265

Closed
Me-Priyank wants to merge 1 commit into
CircuitVerse:masterfrom
Me-Priyank:Priyank-fix
Closed

Fix: contact form focus border color#265
Me-Priyank wants to merge 1 commit into
CircuitVerse:masterfrom
Me-Priyank:Priyank-fix

Conversation

@Me-Priyank
Copy link
Copy Markdown
Member

@Me-Priyank Me-Priyank commented May 31, 2026

Summary

Fixes the contact form focus border color on the /contact page.

Changes

  • Updated the theme primary color override to #40ba84
  • Ensured focused contact form fields use the correct green border
  • Avoided adding !important for this fix

Fixes

Closes #257

Summary by CodeRabbit

  • Chores
    • Improved styling infrastructure for enhanced theme color consistency and maintainability.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 31, 2026

Review Change Stack

Walkthrough

This PR adds a single SCSS variable $primary-color: #40ba84`` to assets/scss/_mixins.scss. The variable defines the CircuitVerse primary green color and enables consistent reuse of this theme value throughout the stylesheet. This change supports the resolution of issue `#257`, which addresses changing the focus border color on the Contact Us form from red to green.

Possibly related PRs

  • CircuitVerse/Blog#256: Also modifies assets/scss/_mixins.scss to reuse the same primary green color (#40ba84) for button hover and active states, directly sharing the theme color value.
🚥 Pre-merge checks | ✅ 5
✅ 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 the contact form focus border color issue referenced in the PR objectives.
Linked Issues check ✅ Passed The PR adds a primary color variable (#40ba84) to address issue #257's requirement to change the contact form focus border from red to green.
Out of Scope Changes check ✅ Passed The change is narrowly scoped: only adds a global SCSS primary color variable directly needed to fix the contact form focus border color as specified in issue #257.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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
Copy Markdown

@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 info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 482cfa4b-3804-46bb-b1ab-9292d5be5a3b

📥 Commits

Reviewing files that changed from the base of the PR and between ee4961f and f6ce60a.

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

Comment thread assets/scss/_mixins.scss
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 31, 2026

✅ Actions performed

Comments resolved and changes approved.

@Nihal4777
Copy link
Copy Markdown
Member

@Me-Priyank Changes applied in #258

@Nihal4777 Nihal4777 closed this May 31, 2026
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 : Change focus border color on Contact Us form from red to green

2 participants