Skip to content

Fix pulse force colors#543

Open
surbhibadhe wants to merge 3 commits into
NextCommunity:mainfrom
surbhibadhe:fix-pulse-force-colors
Open

Fix pulse force colors#543
surbhibadhe wants to merge 3 commits into
NextCommunity:mainfrom
surbhibadhe:fix-pulse-force-colors

Conversation

@surbhibadhe

Copy link
Copy Markdown
Contributor

Updated the pulse button colors for consistent styling.

@deepsource-io

deepsource-io Bot commented Jul 2, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in b0bf6c9...924face 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 Jul 2, 2026 5:31p.m. Review ↗
Secrets Jul 2, 2026 5:31p.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.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request adds a new "PULSE" button to the system override panel and updates corresponding styles in the CSS file, including the "Force Surge" button and the .xp-pulse class. The reviewer pointed out a color inconsistency where the new button uses Tailwind's orange palette while the CSS overrides use amber, and suggested updating the button's Tailwind classes to use amber for consistency.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment on lines +20 to +22
<button onclick="triggerSecretUnlock('pulse')" class="override-btn py-1.5 bg-orange-500/5 hover:bg-orange-500/20 text-orange-400 text-[8px] border border-orange-500/20 rounded transition-all">
PULSE
</button>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The new PULSE button uses Tailwind's orange color palette (bg-orange-500/5, text-orange-400, etc.). However, the corresponding CSS overrides in src/assets/css/style.css use #fbbf24 (Tailwind's amber-400) for the button text and #f59e0b (Tailwind's amber-500) for the .xp-pulse class.

This creates a color mismatch where the button's background and border are orange, but the text is amber/yellow. To maintain consistent styling, the Tailwind classes on this button should be updated to use amber instead of orange.

        <button onclick="triggerSecretUnlock('pulse')" class="override-btn py-1.5 bg-amber-500/5 hover:bg-amber-500/20 text-amber-400 text-[8px] border border-amber-500/20 rounded transition-all">
            PULSE
        </button>

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

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants