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

changed a webp file of logo #375

Closed
wants to merge 6 commits into from
Closed

changed a webp file of logo #375

wants to merge 6 commits into from

Conversation

darshitdudhaiya
Copy link

Copy link
Member

@Akalanka47000 Akalanka47000 left a comment

Choose a reason for hiding this comment

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

Hello,

First off, thanks for the contribution on behalf of the SLIIT FOSS community.
Sadly however both themes seem to break with this fix

  • Dark mode - Missing transparency in the logo background
  • Dark mode - Background purple and navbar purple are two different color codes
  • Light mode - Background should be the original dark gray color

image

image

image

image

@darshitdudhaiya
Copy link
Author

darshitdudhaiya commented Oct 23, 2022 via email

@Akalanka47000
Copy link
Member

Sure😄

@darshitdudhaiya
Copy link
Author

@Akalanka47000 Do you want your logo to be different in dark and light mode? because in this website navbar color and body color are totally different so pls clarify it.

@Akalanka47000
Copy link
Member

@Akalanka47000 Do you want your logo to be different in dark and light mode? because in this website navbar color and body color are totally different so pls clarify it.

Sorry for the late reply, in the dark mode the logo background should be the same color as the background color of the navbar and in the light mode it should be the same dark gray color as it was in the original logo

@Akalanka47000
Copy link
Member

Further there seems to be an outline which has been left during the background removal process😟
image

@Akalanka47000
Copy link
Member

I've updated the format of the .env file as well required for running the project without which you might be stuck. The file is at the root of the project name .env.test. Apologies for the inconvenience.

@Safnaj Safnaj linked an issue Oct 27, 2022 that may be closed by this pull request
@darshitdudhaiya darshitdudhaiya closed this by deleting the head repository Apr 20, 2024
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.

Add Purple BG Logo and set as per the theme.
2 participants