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

FIX : fixed the mesh operator logo bg low opacity in dark mode. #5427

Closed
wants to merge 1 commit into from

Conversation

hunxjunedo
Copy link

@hunxjunedo hunxjunedo commented Feb 29, 2024

Description

This PR fixes #5424. I have introduced new variables in the themes file which are specifically for defining the bg color of mesh operator logo. The logo container also takes the bg from this variable now.

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Signed-off-by: hunx <junaidhunain6@gmail.com>
@l5io
Copy link
Contributor

l5io commented Feb 29, 2024

🚀 Preview for commit 3afa63b at: https://65e0de80d8f3371e521f9a05--layer5.netlify.app

@hunxjunedo
Copy link
Author

@vishalvivekm kindly review the changes.

Copy link

@jameshorton2337 jameshorton2337 left a comment

Choose a reason for hiding this comment

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

Thanks!

However, this doesn't fully solve the issue. See screenshot from https://65e0de80d8f3371e521f9a05--layer5.netlify.app/cloud-native-management/meshery/operating-service-meshes

@hunxjunedo, please know that this is the Meshery Operator version for dark mode, which is available in the layer5.io/brand page.

meshery-operator-white

@leecalcote
Copy link
Member

@VictoryWekwa perhaps, we can do better than this for the dark mode version of the Meshery Operator...

@leecalcote leecalcote requested a review from nwanduka March 1, 2024 21:48
@hunxjunedo
Copy link
Author

Ok so I am changing the logo to properly transparent as it is on the brands page.

@vishalvivekm
Copy link
Member

@hunxjunedo Let's discuss this on Websites's call today 6:30 PM IST. Let's add this as an agenda item into the meeting minutes

@hunxjunedo
Copy link
Author

@vishalvivekm hey sorry I couldn't make it, was busy the whole time.

@iArchitSharma
Copy link
Contributor

@hunxjunedo hey are u still working on it?

@hunxjunedo
Copy link
Author

@iArchitSharma yes, creating a pull request this weekend.

@hunxjunedo
Copy link
Author

@iArchitSharma hey unfortunately im encountering some issues with my system, you can proceed if you want to.

@hunxjunedo
Copy link
Author

@vishalvivekm @jameshorton2337 I was busy for a while. Has there been any progress regarding this issue or should I resolve this as suggested?

@iArchitSharma
Copy link
Contributor

@hunxjunedo Thanks for the concern ,its already fixed

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

Successfully merging this pull request may close these issues.

Operator Icon looks very low opacity
6 participants