Skip to content

Comments

Improve chart readability & fix hover state error#49

Merged
gadomski merged 8 commits intomainfrom
improve-chart-readability
Feb 17, 2026
Merged

Improve chart readability & fix hover state error#49
gadomski merged 8 commits intomainfrom
improve-chart-readability

Conversation

@aboydnw
Copy link
Member

@aboydnw aboydnw commented Feb 16, 2026

Three things in this PR:

  1. I played around with fonts and node positioning and made some slight edits for improved readability
  2. I addressed the hover error in Bug: Hover area after filtering #46
  3. I added two new filters for stars and forks
Screenshot 2026-02-16 at 5 25 49 PM

All the new docs are for a future feature to provide NASA with more visibility into all the repos they are contributing towards.

1. Font size will improve readability of the chart
2. Docs will make it easier for agents to start working, reducing context load
1. filters for stars and forks
2. researched feature request for nasa, including docs for future reference
@aboydnw aboydnw requested a review from gadomski as a code owner February 16, 2026 22:24
@aboydnw aboydnw linked an issue Feb 16, 2026 that may be closed by this pull request
@gadomski
Copy link
Collaborator

All the new docs are for a future feature to provide NASA with more visibility into all the repos they are contributing towards.

Nice! Do you anticipate needing a NASA-specific version of this viz? Or will we just be able to use the "main" version?

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 really like the new filters! The readability is much better as well, and can confirm the hover bug is fixed.

However, I'm 👎🏼 on at least some of the docs. E.g. DECISIONS.md implies that we desinged this app, when in reality its mostly derived from https://github.com/nbremer/ORCA/tree/main/top-contributor-network. What's the minimum you need for the NASA reporting?

@gadomski gadomski linked an issue Feb 17, 2026 that may be closed by this pull request
and remove a few more unnecessary docs
@aboydnw aboydnw requested a review from gadomski February 17, 2026 15:06
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.

Two questions:

  • What's the future value in keeping the plans in the repo?
  • Is there another home for 🤖 docs, other than docs? docs feels like a home for user documentation, not robot documentation.

@aboydnw
Copy link
Member Author

aboydnw commented Feb 17, 2026

I suppose to both questions I don't have strong opinions, it just seemed an easy place to put them to refer agents to in the future. And I assume if it's been useful for me to point agents to those docs then it will be useful for anybody else who comes along and wants to fork the repo and have agents develop on it. But I'm not up to speed on the best practices here, so I'm happy to adjust.

@gadomski
Copy link
Collaborator

But I'm not up to speed on the best practices here, so I'm happy to adjust.

I think we're making them up as we go.

Let's put the claude-specific docs into .claude (including the plans). This mirrors IDE configuration, which goes into (e.g.) .vscode. I'd like to keep things vendor-specific, and not assume everyone uses Claude.

@aboydnw
Copy link
Member Author

aboydnw commented Feb 17, 2026

Sounds good to me! I've been looking up some updated best practices around AGENTS.md files and such to make it more tool agnostic. So we may want to update this in the future. But I think that's a good approach for now.

@aboydnw aboydnw requested a review from gadomski February 17, 2026 17:35
@gadomski gadomski merged commit 8794260 into main Feb 17, 2026
1 check passed
@gadomski gadomski deleted the improve-chart-readability branch February 17, 2026 17:48
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.

Bug: Hover area after filtering Make text more legible

2 participants