Skip to content

Enhance footer branding by enabling clickable logo navigation to homepage#41

Open
Manahil-Afzal wants to merge 5 commits intoOWASP-BLT:mainfrom
Manahil-Afzal:fix/footer-logo
Open

Enhance footer branding by enabling clickable logo navigation to homepage#41
Manahil-Afzal wants to merge 5 commits intoOWASP-BLT:mainfrom
Manahil-Afzal:fix/footer-logo

Conversation

@Manahil-Afzal
Copy link
Copy Markdown
Contributor

@Manahil-Afzal Manahil-Afzal commented Mar 26, 2026

PR Description:
This update improves site navigation by making the footer logo interactive, allowing users to return to the homepage with a single click.

🔧 Changes Implemented:
--- Wrapped the footer logo inside a proper anchor link
--- Configured routing to redirect to the homepage (/)
--- Ensured compatibility with existing layout structure

Summary by CodeRabbit

  • Refactor
    • Made the footer logo/title interactive by converting it to a clickable link that navigates to the site home.

@owasp-blt
Copy link
Copy Markdown

owasp-blt bot commented Mar 26, 2026

📊 Monthly Leaderboard

Hi @Manahil-Afzal! Here's how you rank for March 2026:

Rank User Open PRs PRs (merged) PRs (closed) Reviews Comments Total
35 Neurvinch @Neurvinch 14 2 0 0 2 38
36 Manahil-Afzal @Manahil-Afzal 2 2 2 1 5 33
37 VedantAnand17 @VedantAnand17 0 0 0 5 3 31

Scoring this month (across OWASP-BLT org): Open PRs (+1 each), Merged PRs (+10), Closed (not merged) (−2), Reviews (+5; first two per PR in-month), Comments (+2, excludes CodeRabbit). Run /leaderboard on any issue or PR to see your rank!

@owasp-blt
Copy link
Copy Markdown

owasp-blt bot commented Mar 26, 2026

👋 Hi @Manahil-Afzal!

This pull request needs a peer review before it can be merged. Please request a review from a team member who is not:

  • The PR author
  • coderabbitai
  • copilot

Once a valid peer review is submitted, this check will pass automatically. Thank you!

⚠️ Peer review enforcement is active.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: OWASP-BLT/coderabbit/.coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 5e27ed6a-9e56-456b-8e11-ca3f21b7733f

📥 Commits

Reviewing files that changed from the base of the PR and between 4e449b0 and dd870bc.

📒 Files selected for processing (1)
  • _includes/footer.html

Walkthrough

The footer template was modified to wrap the logo/title section in an anchor tag pointing to the site home, replacing the original non-interactive <div> container. The existing class styling and content structure remain unchanged.

Changes

Cohort / File(s) Summary
Footer Template
_includes/footer.html
Replaced <div> wrapper with <a> tag linking to site home (`{{ '/'

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

quality: low

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: wrapping the footer logo in a clickable link to the homepage, which is exactly what the raw summary and PR objectives document.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

Review ran into problems

🔥 Problems

Timed out fetching pipeline failures after 30000ms


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.

@Manahil-Afzal
Copy link
Copy Markdown
Contributor Author

@DonnieBLT Please Review it

Copy link
Copy Markdown

@Jayant2908 Jayant2908 left a comment

Choose a reason for hiding this comment

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

Looks good!

@owasp-blt owasp-blt bot added has-peer-review PR has received peer review unresolved-conversations: 0 and removed needs-peer-review PR needs peer review unresolved-conversations: 0 labels Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants