OpenFn / OpenFn/lightning

Hardcoded indigo colors bypass the primary-color theme tokens

Open
#4,766 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Elixir
Stars
296
Forks
86
Avg merge
1d 13h
Merged PRs (30d)
50

Description

Summary

app.css defines a theme-aware primary palette (--color-primary-50 through --color-primary-950, currently mapped to indigo), but a number of templates and components hardcode bg-indigo-* / ring-indigo-* / text-indigo-* classes instead of bg-primary-* / ring-primary-* / text-primary-*. If the primary color is ever re-themed, those surfaces stay indigo while the rest of the app shifts.

Where

Currently 23 files, ~58 occurrences. Run grep -rln "bg-indigo\|ring-indigo\|text-indigo" lib/lightning_web/ for the live list. Hotspots:

  • lib/lightning_web/components/pills.ex
  • lib/lightning_web/live/sandbox_live/components.ex
  • lib/lightning_web/live/workflow_live/edit.ex
  • lib/lightning_web/live/project_live/settings.html.heex
  • lib/lightning_web/live/workflow_live/components.ex
  • lib/lightning_web/live/workflow_live/dashboard_components.ex

Fix

Mechanical rename: indigo-NNN -> primary-NNN everywhere a class is part of the primary brand surface. Skip places where indigo is a literal semantic choice (status pills, color picker swatches). One PR.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading app.css to understand the primary color tokens, then run grep -rln "bg-indigo|ring-indigo|text-indigo" lib/lightning_web/ to inspect the 23 affected files and the listed hotspots. Replace indigo classes only on primary brand surfaces, preserving semantic status pills and color-picker swatches, then confirm the resulting class usage across the live list.

Written by the indexing model from the issue text.

Assessment

Tech stack
elixir
Domain
frontend, web-dev
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
74/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.