Fix/minor fixes

[rahul-vyas-dev]

Screenshots/Recordings:

Additional Notes:

• Introduces a new OrgExplorer logo and application icon with a cleaner, more modern design that better represents the project's purpose of exploring GitHub organizations and repository relationships. Also updated the og-image to represent the current logo on social media share.

• Updated the close button color to red.

• Updated the .ico for OrgExplorer, to show latest logo.

• Added line for the yellow graph in the src/pages/AnalyticsPage.jsx.

Checklist

• My code follows the project's code style and conventions
• I have made corresponding changes to the documentation
• My changes generate no new warnings or errors
• I have joined the Discord server and I will share a link to this PR with the project maintainers there
• I have read the Contributing Guidelines

⚠️ AI Notice - Important!

We encourage contributors to use AI tools responsibly when creating Pull Requests. While AI can be a valuable aid, it is essential to ensure that your contributions meet the task requirements, build successfully, include relevant tests, and pass all linters. Submissions that do not meet these standards may be closed without warning to maintain the quality and integrity of the project. Please take the time to understand the changes you are proposing and their impact.

Summary by CodeRabbit

• New Features

  • Added theme-aware logos in the navigation bar and footer.
  • Footer now displays the current year automatically.
  • Updated the app favicon and documentation logo to PNG assets.

• Bug Fixes

  • Repository and contributor health indicators now display correctly in the network visualization.
  • Improved chart styling for created pull requests and issues.

• Style

  • Updated background and text colors for a darker, more consistent appearance.
  • Close buttons in learning and access-token dialogs now use the red accent color.

[coderabbitai]

Walkthrough

The changes update logo references and theme-aware branding, dynamically render the footer year, adjust theme and chart colors, style modal close buttons, and include health scores in repository and contributor network nodes.

Changes

Theme-aware branding and color tokens

Layer / File(s)	Summary
Theme-aware branding and color tokens README.md, index.html, src/components/Navbar.jsx, src/components/layout/Footer.jsx, src/styles/global.css	Logo references now use PNG assets; Navbar and Footer render theme-specific logos, Footer shows the current year, and theme color variables are updated.

Modal and analytics styling

Layer / File(s)	Summary
Modal and analytics styling src/components/LearnModeModal.jsx, src/components/PATModal.jsx, src/pages/AnalyticsPage.jsx	Close buttons use var(--red), while created-series chart strokes and PR fill opacity are adjusted.

Network node health data

Layer / File(s)	Summary
Network node health data src/pages/NetworkPage.jsx	Repository and contributor nodes now include their corresponding healthScore values.

Estimated code review effort: 2 (Simple) | ~10 minutes

Sequence Diagram(s)

sequenceDiagram
  participant ThemeContext
  participant Navbar
  participant Footer
  ThemeContext-->>Navbar: provide current theme
  ThemeContext-->>Footer: provide current theme
  Navbar->>Navbar: render themed logo
  Footer->>Footer: render themed logo and current year

Loading

Possibly related PRs

• AOSSIE-Org/OrgExplorer#72: Introduces the ThemeProvider and useTheme behavior used by these Navbar, Footer, and theme-token changes.
• AOSSIE-Org/OrgExplorer#63: Modifies the NetworkPage health-score visualization logic touched by this change.
• AOSSIE-Org/OrgExplorer#67: Introduces the Footer component updated here with themed branding and dynamic year rendering.

Suggested reviewers: Zahnentferner, bhavik-mangla, Ri1tik

Poem

I hop through themes, both dark and bright,
New little logos shine just right.
Red buttons close with cheerful flair,
Health scores bloom through network air.
The footer counts each passing year—
A rabbit’s review is full of cheer! 🐇

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title is too vague and does not describe the main changes in the pull request.	Use a specific title that reflects the main update, such as adding themed logos/favicon and related UI style fixes.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 3

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@README.md`:
- Line 6: Add a descriptive alt attribute to the logo img element in the README,
identifying it as the organization explorer logo and satisfying markdownlint
MD045.

In `@src/pages/AnalyticsPage.jsx`:
- Line 196: The two Area series in AnalyticsPage should not use hardcoded rgba
fills, which break theme consistency. Update both Area elements, including the
series identified by dataKey "prs_created", to use fill="var(--accent)" with an
appropriate fillOpacity instead of rgba(...) values, while retaining their
existing stroke configuration.

In `@src/pages/NetworkPage.jsx`:
- Around line 36-37: Fix node health propagation in the network construction
logic: use the health-enriched repository source (such as model.totalRepos) or
compute healthScore for repository and contributor data upstream, then store it
on the corresponding data objects consumed by rendering. Remove the ineffective
top-level node healthScore assignment unless rendering functions are updated to
read it; ensure d.data.healthScore is defined for both repo and contributor
nodes and tooltips.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 16871b84-7446-478f-89ba-fb556537e5c1

📥 Commits

Reviewing files that changed from the base of the PR and between d83fe88 and 043dd25.

⛔ Files ignored due to path filters (6)

• package-lock.json is excluded by !**/package-lock.json
• public/og-image.png is excluded by !**/*.png
• public/org-explorer-logo.png is excluded by !**/*.png
• public/org-explorer-logo.svg is excluded by !**/*.svg
• src/assests/og-logo-dark.png is excluded by !**/*.png
• src/assests/og-logo.png is excluded by !**/*.png

📒 Files selected for processing (9)

• README.md
• index.html
• src/components/LearnModeModal.jsx
• src/components/Navbar.jsx
• src/components/PATModal.jsx
• src/components/layout/Footer.jsx
• src/pages/AnalyticsPage.jsx
• src/pages/NetworkPage.jsx
• src/styles/global.css