Skip to content

Update CNCF report chart titles for clarity#1761

Merged
jonathimer merged 1 commit intomainfrom
fix/cncf-report-chart-titles
Mar 20, 2026
Merged

Update CNCF report chart titles for clarity#1761
jonathimer merged 1 commit intomainfrom
fix/cncf-report-chart-titles

Conversation

@jonathimer
Copy link
Collaborator

Summary

  • Updates subtitle to "Geographical distribution of contributing organizations across all CNCF projects"
  • Updates chart heading to "Geographical Distribution Over Time"

Test plan

  • Verify text changes render correctly on /report/cncf page

Signed-off-by: Jonathan Reimer <jonathan@reimer.me>
Copilot AI review requested due to automatic review settings March 20, 2026 18:47
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

PR titles must follow Conventional Commits. Love from, Your reviewers ❤️.

@jonathimer jonathimer merged commit 7718805 into main Mar 20, 2026
9 of 10 checks passed
@jonathimer jonathimer deleted the fix/cncf-report-chart-titles branch March 20, 2026 18:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates UI copy on the CNCF report page to make chart titles/subtitles clearer for users viewing geographic distribution data.

Changes:

  • Updated the page subtitle text under the main heading.
  • Updated the “over time” chart section heading.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

</h1>
<p class="text-body-2 md:text-body-1 text-neutral-600">
Geographical distribution of contributors across all CNCF projects over time
Geographical distribution of contributing organizations across all CNCF projects
Copy link

Copilot AI Mar 20, 2026

Choose a reason for hiding this comment

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

The subtitle now says "contributing organizations", but this view (and the backing Tinybird pipes) appears to be showing contributors (e.g., uses contributorCount throughout and fetches from collection_contributors_geo_distribution*). This copy change could be misleading—please align the subtitle terminology with what the data represents (contributors vs organizations).

Suggested change
Geographical distribution of contributing organizations across all CNCF projects
Geographical distribution of contributors across all CNCF projects

Copilot uses AI. Check for mistakes.
</h1>
<p class="text-body-2 md:text-body-1 text-neutral-600">
Geographical distribution of contributors across all CNCF projects over time
Geographical distribution of contributing organizations across all CNCF projects
Copy link

Copilot AI Mar 20, 2026

Choose a reason for hiding this comment

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

The page title uses "Geographic" while the updated subtitle uses "Geographical". Since they appear together, it reads inconsistent—consider standardizing to one form (e.g., "Geographic" everywhere).

Suggested change
Geographical distribution of contributing organizations across all CNCF projects
Geographic distribution of contributing organizations across all CNCF projects

Copilot uses AI. Check for mistakes.
<div class="flex flex-col gap-3 sm:flex-row sm:items-center sm:justify-between mb-4">
<h2 class="text-body-1 md:text-heading-3 font-secondary font-semibold text-neutral-900">
Contributor Distribution Over Time
Geographical Distribution Over Time
Copy link

Copilot AI Mar 20, 2026

Choose a reason for hiding this comment

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

This heading uses "Geographical" while the main page title above uses "Geographic". For consistency in UI copy, consider using the same wording in both places.

Suggested change
Geographical Distribution Over Time
Geographic Distribution Over Time

Copilot uses AI. Check for mistakes.
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