fix(minor UI): fixes the inconsistent theme of tooltip in dark/light mode. Replace hardcoded colors with theme tokens in chart copy button tooltip #8972
+1
−1
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR fixes the tooltip color inconsistency in dark and light modethat happens on any barchart copy icon tooltip. hence, refactors the
ChartCopyButtoncomponent to use theme-aware color tokens instead of hardcoded colors for better theming support.Changes
refactor(components): replace hardcoded colors with theme tokens in chart copy button tooltip
Changed the
TooltipContentclassName frombg-black text-whitetobg-foreground text-backgroundin theChartCopyButtoncomponent. This ensures the tooltip respects the current theme and provides better consistency across light and dark modes.Files Changed
apps/v4/components/chart-copy-button.tsxType of Change
Testing
Screenshots
before fix:
after fix: