Conversation
Replace custom Roboto/Oswald fonts and blue/pink/orange palette with FalkorDB standard: Inter font, purple primary accent, clean dark/light mode with theme toggle, and consistent shadcn/ui component usage. - Update CSS variables to purple primary, add full dark mode token set - Rename graph category colors with graph- prefix to separate from theme - Add ThemeProvider (system/light/dark, localStorage) and ThemeToggle - Redesign header: compact layout, icon-only nav with tooltips, remove gradient bar - Replace all hardcoded colors (bg-white, text-black, bg-gray-*, etc.) with semantic tokens (bg-background, text-foreground, bg-muted, etc.) - Standardize buttons/inputs to shadcn Button/focus-ring patterns - Add dark scrollbar styles and panel resize hover states Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Align UI with FalkorDB design language
Make the graph canvas and related loading states follow the active theme so dark mode no longer shows bright mismatched surfaces. Validation: - npm --prefix ./app run lint - npm --prefix ./app run build Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…20260314-1506 Fix dark mode graph surfaces
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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. Comment |
There was a problem hiding this comment.
Pull request overview
This PR introduces system-aware theming (light/dark) across the React UI, updates styling to use theme tokens, and improves Playwright E2E stability for canvas/graph interactions to support the updated UI.
Changes:
- Add
ThemeProvider+ThemeToggle, wire provider at app root, and update UI components to use theme-based Tailwind tokens. - Update graph canvas rendering to adapt background/foreground and dimmed-node colors to the active theme.
- Improve E2E graph selection and canvas interactions (more deterministic waits, updated locators, better coordinate handling).
Reviewed changes
Copilot reviewed 17 out of 17 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| e2e/logic/POM/codeGraph.ts | More robust graph/canvas waiting + locator updates aligned with UI changes |
| app/tailwind.config.ts | Add Inter font + rename custom graph colors + theme token colors |
| app/src/main.tsx | Wrap app with ThemeProvider |
| app/src/globals.css | Theme token variable tweaks + replace hardcoded styles with tokens |
| app/src/components/theme-toggle.tsx | New theme toggle button |
| app/src/components/theme-provider.tsx | New theme context/provider with system theme support + persistence |
| app/src/components/model.ts | Update category color naming to match Tailwind config |
| app/src/components/labels.tsx | Adjust background to theme token |
| app/src/components/graphView.tsx | Theme-aware canvas colors + dimmed-node styling |
| app/src/components/elementMenu.tsx | Theme token styling + icon color inheritance |
| app/src/components/dataPanel.tsx | Theme token styling + transparent code block backgrounds |
| app/src/components/combobox.tsx | Theme token borders + focus ring |
| app/src/components/code-graph.tsx | Theme token layout + replace some buttons with UI Button |
| app/src/components/chat.tsx | Theme token styling + replace loading gif with spinner UI |
| app/src/components/Input.tsx | Theme token styling for inputs/list |
| app/src/App.tsx | Navbar redesign (icons + tooltips) + add theme toggle (desktop + mobile) |
| app/index.html | Limit Google Fonts import to Inter only |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Provide architecture, commands, conventions, environment variables, testing, and API endpoint reference for AI-assisted development. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Validate localStorage theme value against allowed themes before use, falling back to defaultTheme on mismatch or missing value - Create ThemeProviderContext with undefined default so useTheme guard properly detects usage outside ThemeProvider - Add aria-label to icon-only theme toggle button for screen readers - Standardize 'Github' key to 'GitHub' in e2e test selectors Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Add 'text' language tag to directory tree fenced block (MD040) - Clarify async-first convention to acknowledge sync helpers in api/graph.py Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add AGENTS.md project guide and CLAUDE.md symlink
…Color Replace <img> tags loading static SVG files (with hardcoded dark fills) with inline SVG React components that use `currentColor` for theme-adaptive fills. The gradient swirl remains unchanged. - Create Logo component (desktop header F icon) - Create CodeGraphLogo component (mobile header full logo) - Update App.tsx to use inline SVG components Co-authored-by: gkorland <753206+gkorland@users.noreply.github.com>
- Remove code-graph-logo.tsx component (unused after change) - Mobile header now uses Logo + 'CODE GRAPH' text, matching desktop layout - Fix duplicate SVG gradient ID conflict using React useId() - Change mobile header from absolute-positioned icons to flex justify-between Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Fix logo visibility in dark mode using inline SVG with currentColor
The logo was refactored from an <img> tag to an inline SVG <Logo /> component, but the E2E POM selector still looked for img[@alt='FalkorDB']. Updated to use the aria-label attribute on the parent <a> tag instead. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
No description provided.