Skip to content

feat: add modern brand logo to navbar, footer, and favicon (resolves … #178

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
mahakagarwal7 wants to merge 1 commit into
base: main
Choose a base branch
from
+6 −4
Open

feat: add modern brand logo to navbar, footer, and favicon (resolves … #178

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion frontend/index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
<html lang="en">
<head>
<meta charset="UTF-8" />
<link rel="icon" type="image/svg+xml" href="/favicon.svg" />
<link rel="icon" type="image/png" href="/logo.png" />
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<title>CodeLens</title>
</head>
Expand Down
Binary file added frontend/public/logo.png
View file
Open in desktop
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
5 changes: 3 additions & 2 deletions frontend/src/components/shared/Footer.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,9 +14,10 @@ export default function Footer() {
<div className="md:col-span-2 xl:col-span-1 flex flex-col gap-6 border-b-4 border-black pb-10 md:border-b-0 md:pb-0 md:border-r-4 md:pr-10 xl:border-r-4 xl:pr-8">
<Link
to="/"
className="text-5xl font-black tracking-tighter uppercase text-black leading-none hover:opacity-70 transition-opacity"
className="flex items-center gap-4 text-5xl font-black tracking-tighter uppercase text-black leading-none hover:opacity-70 transition-opacity"
>
CODE<br />LENS
<img src="/logo.png" alt="CodeLens Logo" className="h-16 w-auto" />
<div>CODE<br />LENS</div>
</Link>
<p className="text-sm font-bold tracking-widest uppercase leading-relaxed text-black max-w-xs">
The developer intelligence platform. Unify your competitive programming journey across Codeforces, LeetCode &amp; GitHub — with AI-driven insights, zero noise.
Expand Down
3 changes: 2 additions & 1 deletion frontend/src/components/shared/Navbar.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -373,8 +373,9 @@ export default function Navbar() {
<Link
to="/"
onClick={closeMenu}
className="text-lg font-black tracking-tighter uppercase text-black hover:opacity-60 transition-opacity flex-shrink-0"
className="flex items-center gap-2 text-lg font-black tracking-tighter uppercase text-black hover:opacity-60 transition-opacity flex-shrink-0"
>
<img src="/logo.png" alt="CodeLens Logo" className="h-6 w-auto" />
CODELENS
Comment on lines +378 to 379

@coderabbitai coderabbitai Bot Jun 10, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Consolidated: duplicate accessible name on decorative brand logos.

Both brand links already expose textual labels, so logo alt text should be empty in both places to prevent repeated screen-reader output. Standardize on alt="" aria-hidden="true" for these decorative instances.

🤖 Prompt for 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.

In `@frontend/src/components/shared/Navbar.jsx` around lines 378 - 379, The
decorative brand logo image in Navbar.jsx currently has a non-empty alt
("CodeLens Logo") which duplicates the visible text label; update the <img> used
in the Navbar component (the logo instance with src "/logo.png") to be purely
decorative by setting alt="" and adding aria-hidden="true" so screen readers do
not read the logo twice; apply the same change to the other logo instance in
this component if present to standardize behavior.

</Link>

Expand Down
Loading