Skip to content

Conversation

@srivastava-diya
Copy link
Contributor

Description

This PR fixes an issue where the active tab text color on the white-labeling page did not match the Meshery design theme. It updates the SASS styles so that the active state now reflects the correct primary text color.

Fixes Issue

Fixes #818

Notes for Reviewers

Please verify that:
-The active tab text color now uses the proper theme variable.

Screenshots

Before:

Before: white-labeling active tab issue

After:

After: white-labeling active tab fixed

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Diya <diyasrivastava2023@gmail.com>
@netlify
Copy link

netlify bot commented Nov 6, 2025

Deploy Preview for bejewelled-pegasus-b0ce81 ready!

Name Link
🔨 Latest commit 0c100b2
🔍 Latest deploy log https://app.netlify.com/projects/bejewelled-pegasus-b0ce81/deploys/690d2edab6a51c0007bca2ee
😎 Deploy Preview https://deploy-preview-850--bejewelled-pegasus-b0ce81.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@srivastava-diya
Copy link
Contributor Author

hey @Namanv0509 this PR addresses issue #818 . Kindly check it out.

@leecalcote leecalcote merged commit 4f4a4b9 into layer5io:master Nov 12, 2025
6 checks passed
@leecalcote
Copy link
Member

Thanks! We could improve upon this slightly by using a different background color for the active versus inactive tabs and/or different text color for the inactive/active tabs.

@srivastava-diya
Copy link
Contributor Author

srivastava-diya commented Nov 12, 2025

Thanks! We could improve upon this slightly by using a different background color for the active versus inactive tabs and/or different text color for the inactive/active tabs.

yes @leecalcote we can do this, the inactive tab color could be changed to white or any other suggestion that you would want to be implemented

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[UI] White labeling page active tab text color fix

2 participants