Skip to content

Conversation

@aboydnw
Copy link
Member

@aboydnw aboydnw commented Feb 9, 2026

Another cleanup to remove some unnecessary code. This used to be a visual element for a center node in the middle of the chart that I removed to make it visually cleaner. From here I think we can start playing with things like font size and filtering to make the visual easier to scan.

To test:

  1. Make sure chart elements and interactions still function
  2. Make sure title still says "Development Seed"
Screenshot 2026-02-09 at 7 54 48 AM Screenshot 2026-02-09 at 7 54 29 AM

@aboydnw aboydnw requested a review from gadomski as a code owner February 9, 2026 15:55
@aboydnw
Copy link
Member Author

aboydnw commented Feb 9, 2026

I reordered the contributors alphabetically because the symmetry of the chart in the screenshot above was annoying me 😅 so here is an updated screenshot

Screenshot 2026-02-09 at 10 43 03 AM

Also, something we get for free with this change is that when you apply a filter the chart simplifies to only show contributors with links to the filtered repos. We got this for free on this PR because now there is no central node creating a "fake link." I'm okay with this functionality for now, although open to feedback. In future stories I want to work on the filtering functionality so we can improve this interaction then.

Screenshot 2026-02-09 at 10 45 19 AM

Copy link
Collaborator

@gadomski gadomski left a comment

Choose a reason for hiding this comment

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

I like it! I don't know if it happened with this PR or previously, but mousing over a indvidual's circle doesn't appear to give them a popup anymore. I'm cool with merging this and fixing that as a follow-on.

Screen.Recording.2026-02-10.at.3.55.19.AM.mov

@aboydnw aboydnw merged commit 333695e into main Feb 10, 2026
1 check passed
@aboydnw aboydnw deleted the remove-center-node branch February 10, 2026 16:42
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