Skip to content

fix: make header logo clickable on stats and contributors pages#210

Draft
github-actions[bot] wants to merge 1 commit into
mainfrom
fix/standardize-header-logo-175-8e512f5e50835124
Draft

fix: make header logo clickable on stats and contributors pages#210
github-actions[bot] wants to merge 1 commit into
mainfrom
fix/standardize-header-logo-175-8e512f5e50835124

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

This PR standardizes the header logo markup across the docs pages and makes it clickable for improved navigation.

Changes

  • ✅ Made the logo clickable on stats.html and contributors.html pages
  • ✅ Logo now links back to index.html (the main jobs page)
  • ✅ Added aria-label="New Grad Jobs home" for accessibility
  • ✅ Kept the logo static on index.html (since it's already on the home page)
  • ✅ Updated CHANGELOG.md with this change

Testing

Manually verified:

  • Logo structure is consistent across all three pages
  • Logo is clickable on stats and contributors pages
  • Logo correctly links to index.html
  • Existing CSS classes preserved (no visual changes)
  • Accessibility maintained with proper aria-label

Fixes #175

Generated by Daily Backlog Burner ·

- Wrap logo in anchor tag linking to index.html on stats and contributors pages
- Add aria-label for accessibility
- Keep logo static on index.html (current page)
- Update CHANGELOG.md with this change

Fixes #175

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 19, 2026

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 2a7b4217-bf7b-4cb5-9cb3-e1195111efb6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/standardize-header-logo-175-8e512f5e50835124
📝 Coding Plan
  • Generate coding plan for human review comments

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

Comment @coderabbitai help to get the list of available commands and usage tips.

@ambicuity
Copy link
Copy Markdown
Owner

Blocked remediation summary:

Current blockers:

  • draft PR
  • mergeStateStatus: DIRTY (conflicts)
  • superseded by merged PR #220 for header-logo standardization

Recommendation:

  • Close as superseded unless there is unique behavior not included in #220.

@github-actions github-actions Bot added the merge-conflict This PR has merge conflicts that must be resolved label Mar 26, 2026
@github-actions
Copy link
Copy Markdown
Contributor Author

👋 Hi @github-actions[bot]! This PR has a merge conflict with main and can't be merged until it's resolved.

To fix it:

git fetch origin
git rebase origin/main
# Resolve any conflicts, then:
git add .
git rebase --continue
git push --force-with-lease

This label will be removed automatically once the conflict is resolved. 🙏

@ambicuity
Copy link
Copy Markdown
Owner

Triage refresh (2026-04-08):\n- Status: Draft + merge conflicts; earlier maintainer note marks it superseded by merged #220.\n- Linked issue: #175 is CLOSED; this PR is likely redundant unless it contains distinct behavior.\n- AI reviewer relevance: CodeRabbit output is skip/status only (bot-detected), no actionable findings.\n- Recommendation: keep documented-only and close as superseded if no unique delta exists.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated backlog-burn merge-conflict This PR has merge conflicts that must be resolved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Good First Issue]: standardize header logo markup across docs pages

1 participant