Skip to content
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.

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

#591 Enhance Dark Mode Experience: Add a New Logo for Improved Visibility #593

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ianjiteshan
Copy link

🚀 Pull Request

Description

In this pull request, I added two new logo images: cabz.png (light mode) and cabz-dark.png (dark mode). I updated the image sources in the code to switch between these logos based on the theme (light or dark mode).

Fixes # (issue)

  • Added two new images (cabz.png and cabz-dark.png) in the /img folder.
  • Updated the src attributes of the logo images in the code to reflect the theme switch.
  • Ensured that the dark mode logo (cabz-dark.png) appears when dark mode is enabled, and the light mode logo (cabz.png) appears when dark mode is disabled.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update

Checklist:

  • I have added the new images to the /img folder.
  • I have correctly updated the src attributes in the code where required.
  • I have tested the logo switching functionality in both light and dark mode.
  • I have verified that both logos are displayed correctly based on the current theme.
  • I have ensured no other functionality was broken in the process.
  • My changes generate no new warnings or errors in the console.
  • I have added relevant comments or explanations where necessary in the code.
  • Screenshots of the final output are attached.

Screenshot of final output/video

Light Mode Logo
Dark Mode Logo

@ianjiteshan
Copy link
Author

@07sumit1002 pls check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant