Updated the header color and moved away from yellow shade to more dar… #2104
Conversation
dosubot
bot
added
the
size:S
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
PR Summary
Theme update that establishes a more professional look by replacing the yellow/golden accent colors with a darker color scheme (#14151a) in the documentation header and navigation elements.
- Changed header border color from golden (#bd8526) to dark gray (#14151a) in
docs/_static/js/header_border.js - Added white background (#ffffff) for light mode in
docs/extra/ragas-modern.css - Implemented consistent dark header/tab navigation background across themes
- Removed yellow/orange divider elements from header for cleaner appearance
2 files reviewed, 3 comments
Edit PR Review Bot Settings | Greptile
| @@ -1,6 +1,6 @@ | |||
| const header_div = document.querySelector(".md-header"); | |||
| const navbar_div = document.querySelector(".md-tabs"); | |||
| const border_css = "2px solid #bd8526"; | |||
| const border_css = "2px solid #14151a"; | |||
There was a problem hiding this comment.
style: Consider using a CSS variable for the border color instead of hardcoding it. This would make theme changes easier to manage across the entire UI.
| /* Header background color for both light and dark modes */ | ||
| .md-header { | ||
| background-color: #14151a !important; | ||
| } |
There was a problem hiding this comment.
style: Using !important here overrides Material Design's theme system. Consider using CSS custom properties or theme variables instead for better maintainability
| .md-tabs { | ||
| background-color: #14151a !important; | ||
| } |
There was a problem hiding this comment.
style: Duplicate !important usage. Consider consolidating header and tab styles into a single ruleset since they share the same color
…ker one