Skip to content

Conversation

Bhup-GitHUB
Copy link
Contributor

@Bhup-GitHUB Bhup-GitHUB commented Sep 13, 2025

What has changed?

Fixed dark mode styling issues in the InstallReminder component where the white card with light gray text had poor contrast and readability in dark theme.

This PR Resolves #2986

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 (if none of the other choices apply).

How Has This Been Tested?

Please run npm run build and npm run serve to check if the changes are working as expected. Please include screenshots of the output of both the commands. Add screenshots/gif of the changes if possible.
image

Build

image

Checklist:

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.

Changes Made:

  • Replaced hard-coded colors with CSS variables for theme compatibility
  • Updated card background to use var(--ifm-card-background-color)
  • Fixed text colors to use theme-aware variables for proper contrast
  • Improved visual hierarchy with better spacing

Signed-off-by: Bhupesh Kumar <[email protected]>
Copy link
Contributor

@Achanandhi-M Achanandhi-M left a comment

Choose a reason for hiding this comment

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

Hey @Bhup-GitHUB,

Could you please separate them and also add a clear title and description? At the moment, it’s a bit difficult to understand what this PR is about.

@Bhup-GitHUB
Copy link
Contributor Author

Hey @Bhup-GitHUB,

Could you please separate them and also add a clear title and description? At the moment, it’s a bit difficult to understand what this PR is about.

separate what ?

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.

2 participants