Skip to content

fix banner colors#2854

Merged
atharvadeosthale merged 1 commit intomainfrom
atharva-banner-fix
Apr 3, 2026
Merged

fix banner colors#2854
atharvadeosthale merged 1 commit intomainfrom
atharva-banner-fix

Conversation

@atharvadeosthale
Copy link
Copy Markdown
Member

No description provided.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 3, 2026

Greptile Summary

This PR fixes the MongoDB top-banner color scheme by swapping the background and text colors. Previously, the banner rendered with a dark navy background (#001e2b) and bright green text (#00ed64), which was the inverse of the intended design described by the existing comment ("bright leaf green bar, dark navy copy"). The fix produces a bright leaf-green background with dark navy text, matching MongoDB's brand palette.

  • Background changed from dark navy #001e2b to bright leaf green #00ed64
  • Text color changed from bright leaf green #00ed64 to dark navy #001e2b
  • No functional logic, layout, or other styles are affected
  • The inline comment at line 15 now accurately describes the resulting appearance

Confidence Score: 5/5

This PR is safe to merge — it is a minimal, self-contained color swap with no logic changes.

Only two CSS property values are changed, both are intentional (matching the pre-existing inline comment), and no other styles or components are affected. No P0 or P1 issues found.

No files require special attention.

Important Files Changed

Filename Overview
src/scss/7-components/_top-banner.scss Swaps MongoDB banner background (#001e2b → #00ed64) and text color (#00ed64 → #001e2b) so the visual matches the existing comment ("bright leaf green bar, dark navy copy").

Reviews (1): Last reviewed commit: "fix banner colors" | Re-trigger Greptile

@atharvadeosthale atharvadeosthale merged commit 8826583 into main Apr 3, 2026
6 checks passed
@atharvadeosthale atharvadeosthale deleted the atharva-banner-fix branch April 3, 2026 11:30
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.

2 participants