Skip to content

Conversation

Copy link

Copilot AI commented Dec 5, 2025

Graph edges were nearly invisible on load due to ReactFlow's default light gray stroke color (#b1b1b7) lacking contrast against light backgrounds.

Changes

  • src/index.css: Override ReactFlow CSS variables

    • Light mode: --xy-edge-stroke-default: #111827 (dark stroke, 2px width)
    • Dark mode: --xy-edge-stroke-default: #e5e7eb (light stroke, 2px width)
    • Selected: --xy-edge-stroke-selected-default: #ef4444 (red highlight)
  • src/main.jsx: Import index.css to apply overrides

.react-flow {
  --xy-edge-stroke-default: #111827;
  --xy-edge-stroke-width-default: 2;
  --xy-edge-stroke-selected-default: #ef4444;
}

.react-flow.dark {
  --xy-edge-stroke-default: #e5e7eb;
  --xy-edge-stroke-width-default: 2;
  --xy-edge-stroke-selected-default: #ef4444;
}

Screenshots

Before: Edges barely visible
before

After: Clear edge visibility
after-models
after-concepts

Original prompt

when the graph first loads in some themes - the lines cannot be seen

Created from VS Code via the GitHub Pull Request extension.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits December 5, 2025 17:59
Co-authored-by: grovesy <62599442+grovesy@users.noreply.github.com>
Co-authored-by: grovesy <62599442+grovesy@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix visibility issue for graph lines in certain themes Override ReactFlow CSS variables for visible edge strokes Dec 5, 2025
Copilot AI requested a review from grovesy December 5, 2025 18:34
@grovesy
Copy link
Owner

grovesy commented Dec 5, 2025

@copilot the dos theme should be fixed width fonts

Copy link
Author

Copilot AI commented Dec 5, 2025

@copilot the dos theme should be fixed width fonts

Could you clarify which elements should use fixed-width fonts? I see:

  • Concept file viewer already uses fontFamily: 'monospace'
  • Table view fields/types use monospace
  • UML node fields use monospace

The CSS classes in index.css (.sidebar, .main-content) aren't currently used - the app uses Material-UI components. Should I:

  1. Update the MUI theme to use monospace globally?
  2. Add monospace to specific components?
  3. Something else?

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