Skip to content

fix: health score report LF/non-LF color convention IN-1304 - #2205

Merged
gaspergrom merged 2 commits into
mainfrom
fix/IN-1304-color-convention
Sep 16, 2026
Merged

gaspergrom merged 2 commits into
mainfrom
fix/IN-1304-color-convention

Conversation

@gaspergrom

@gaspergrom gaspergrom commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

Applies the report-wide Linux-Foundation-vs-other-projects color convention (brand blue for LF, violet for non-LF, consistently) to the "Signal availability inside and outside the Linux Foundation" widget (signal-availability-lf.vue). It previously colored the Linux Foundation bars per-category — matching the unrelated signal-scores widget's category-color scheme — with the "Other" series always neutral gray, which read as inconsistent for a widget whose entire point is an LF-vs-other comparison, not a category comparison. No chart type change for this widget.

Known/accepted limitation

This widget still shows a binary Linux-Foundation-vs-Other comparison rather than a full distribution, because the underlying Tinybird pipe only returns one aggregate percentage per scope. A true distribution chart would need a backend/pipe change, out of scope for this frontend-only pass.

Test plan

  • pnpm tsc-check — passes
  • pnpm lint:fix — 0 errors (pre-existing unrelated warnings only)
  • pnpm test — all assertions pass
  • Manually verified in the running app: all three category groups (Maintainer health, Security & supply chain, Development activity) render Linux Foundation bars in blue and Other bars in purple consistently, legend updated to match, tooltips unaffected

Deploy order

N/A — frontend-only change, no backend/pipe dependency.

Migration checklist

N/A — no database changes.

Notes

Independent of the other three PRs in this set (copy/structure #2203, chart conversions #2204) — targets release/IN-1304-health-score-report-feedback directly since it only touches this one widget's colors and has no shared-file overlap with the others.

  • release/IN-1304-health-score-report-feedback - ⚠️ No PR associated with branch
    • fix: health score report LF/non-LF color convention IN-1304 👈

Applies the report-wide Linux-Foundation-vs-other-projects color
convention (brand blue for LF, violet for non-LF) to the "Signal
availability inside and outside the Linux Foundation" widget. It
previously colored the Linux Foundation series per-category (matching
the neighboring signal-scores widget's category colors) with the
"Other" series always gray, which read as inconsistent for a widget
that is specifically an LF-vs-other comparison rather than a
category comparison. No chart type change.

Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
Copilot AI balanced review requested due to automatic review settings September 15, 2026 12:02

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Standardizes LF-vs-other colors in the signal availability widget.

Changes:

  • Uses brand blue for LF and violet for other projects.
  • Updates legend colors and removes category-color indicators.
  • Simplifies chart configuration.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +126 to +128
// Report-wide LF-vs-non-LF color convention: Linux Foundation projects always blue, other tracked
// projects always purple, consistent across every category group (this widget IS an LF-vs-other
// comparison, unlike widget 03's Top-20%-vs-median comparison, which keeps its own category colors).
@gaspergrom
gaspergrom changed the base branch from release/IN-1304-health-score-report-feedback to main September 15, 2026 13:08
@gaspergrom
gaspergrom requested a review from epipav September 15, 2026 15:28
@gaspergrom gaspergrom self-assigned this Sep 15, 2026
Copilot AI review requested due to automatic review settings September 16, 2026 01:46
@gaspergrom
gaspergrom merged commit 1303080 into main Sep 16, 2026
10 checks passed
@gaspergrom
gaspergrom deleted the fix/IN-1304-color-convention branch September 16, 2026 01:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

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.

3 participants