feat: add modern brand logo to navbar, footer, and favicon (resolves …#178
feat: add modern brand logo to navbar, footer, and favicon (resolves …#178mahakagarwal7 wants to merge 1 commit into
Conversation
|
@mahakagarwal7 is attempting to deploy a commit to the Kunal Verma's projects Team on Vercel. A member of the Team first needs to authorize it. |
📝 WalkthroughWalkthroughThe PR integrates a new logo asset ( ChangesBranding Asset Integration
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Suggested labels
Poem
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
🚀 PR Received SuccessfullyHello @mahakagarwal7, Thank you for taking the initiative to contribute to this project. Please ensure that your PR follows all project guidelines properly before requesting review.
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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 `@frontend/src/components/shared/Navbar.jsx`:
- Around line 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.
🪄 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: defaults
Review profile: CHILL
Plan: Pro
Run ID: f3e7c7ba-e921-47c8-abed-5c613e568285
⛔ Files ignored due to path filters (1)
frontend/public/logo.pngis excluded by!**/*.png
📒 Files selected for processing (3)
frontend/index.htmlfrontend/src/components/shared/Footer.jsxfrontend/src/components/shared/Navbar.jsx
| <img src="/logo.png" alt="CodeLens Logo" className="h-6 w-auto" /> | ||
| CODELENS |
There was a problem hiding this comment.
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.
|
@mahakagarwal7 Good I want if you can come up with some more sample logos , atleast 10 Welcome to Codelens and this Work on making logo is going to be presented as first view as every user will going to see logo very first in their journey while visiting the site By the way one more contributor is working on this so best one will going to be choosed Looking for your contribution |
2 participants
This PR resolves #24 by introducing a modern, premium brand logo and integrating it across the frontend application.
The new visual identity was designed to perfectly compliment the platform's brutalist, high-contrast aesthetic. It features a stark, abstract monogram ("C-L") that subtly forms a camera lens and coding brackets, giving the project a top-tier tech startup feel.
Changes Made:-
1)Asset Creation: Added the new logo.png (transparent background, high-resolution flat vector) to frontend/public/logo.png.
2)Favicon Update: Updated frontend/index.html to reference the new logo as the browser favicon.
3)Navbar Integration: Updated Navbar.jsx to render the new logo directly next to the CODELENS wordmark.
4)Footer Integration: Updated the main brand block in Footer.jsx to include the new logo.
Summary by CodeRabbit
Style