Skip to content

Added Custom Branding Browser Favicon Logo Support#262

Open
Dhruv-kundu wants to merge 2 commits into
utksh1:mainfrom
Dhruv-kundu:Add-Logo
Open

Added Custom Branding Browser Favicon Logo Support#262
Dhruv-kundu wants to merge 2 commits into
utksh1:mainfrom
Dhruv-kundu:Add-Logo

Conversation

@Dhruv-kundu
Copy link
Copy Markdown

Related Issue
Closes #197

program: gssoc

Description

  • This PR enhances the branding and visual identity of the SecuScan | Executive Security Briefing project by adding favicon support for the browser tab.

Type of Change

  • New feature
  • UI enhancement

Changes Made
-- Browser Tab Favicon

  • Added custom favicon support using:

Benefits

  • Better project branding
  • Cleaner and more professional UI
  • Improved browser tab identification
  • Responsive and lightweight implementation

Checklist

  • Code follows project style guidelines
  • Tested changes locally
  • No console errors introduced
  • Responsive design maintained
  • PR is linked to the related issue

Screenshots

  • Added favicon in browser tab
    Before
Screenshot 2026-05-22 144822

After
Screenshot 2026-05-23 092211

Testing

  • Verified favicon visibility in browser tab
  • Checked responsiveness across different screen sizes

Logo Added
favicon

@Dhruv-kundu
Copy link
Copy Markdown
Author

Hello @utksh1 , Myself Dhruv .
I have added a custom favicon logo for SecuScan | Executive Security Briefing 🔐
Changes Made

  • Designed and integrated a new security-themed favicon
  • Added branding aligned with the project’s executive security aesthetic
  • Improved visual identity and browser tab appearance
    Please review the updated favicon integration and let me know if any refinements are needed.

@utksh1 utksh1 added area:frontend Frontend React/UI work type:design Design or UX work category bonus label level:beginner 20 pts difficulty label for small beginner-friendly PRs labels May 23, 2026
Copy link
Copy Markdown
Owner

@utksh1 utksh1 left a comment

Choose a reason for hiding this comment

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

Thanks for the branding contribution. I am requesting changes before this can merge.

Why this is not merge-ready yet:

  • The favicon file is very large for a browser tab icon: frontend/favicon.png is 1024x1024 and about 1.4 MB. A favicon should be optimized and tiny, usually 32x32/48x48 or a small .ico/PNG set.
  • The image includes detailed text (SecuScan and the subtitle). That text will be unreadable at favicon size, so the favicon should use a simplified icon-only mark.
  • The PR says Closes #197, but #197 asks for broader logo/branding integration across README/docs/UI. This PR only adds browser favicon support, so either complete the requested integration or change the PR wording so it does not close #197.
  • Please place static favicon assets in the frontend static/public path expected by the Vite build, and verify the built app resolves the icon correctly.

Labels applied: area:frontend, type:design, level:beginner. I did not add gssoc:approved because this needs changes first.

@Dhruv-kundu
Copy link
Copy Markdown
Author

Dhruv-kundu commented May 23, 2026

Hello @utksh1 . Myself Dhruv .
Issue closes:#197
pr : #262
I have made the changes you asked for . And my issue is updated to favicon and pr is already for it .
favicon
Screenshot 2026-05-23 104703

png
Favicon-SecuScan

Readme update
Screenshot 2026-05-23 104715

Requested label
type:docs
type:refactor

If like my work than can add label quality:clean

@Dhruv-kundu Dhruv-kundu requested a review from utksh1 May 23, 2026 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:frontend Frontend React/UI work level:beginner 20 pts difficulty label for small beginner-friendly PRs type:design Design or UX work category bonus label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Integrate Official Project Favicon logo

2 participants