Skip to content

Scaling: Make chart colors responsive#376

Merged
sniok merged 1 commit into
Azure:mainfrom
skoeva:scaling
Mar 9, 2026
Merged

Scaling: Make chart colors responsive#376
sniok merged 1 commit into
Azure:mainfrom
skoeva:scaling

Conversation

@skoeva
Copy link
Copy Markdown
Collaborator

@skoeva skoeva commented Mar 6, 2026

These changes replace hardcoded colors in ScalingChart with MUI theme tokens so the chart renders correctly in light/dark mode.

  • Tooltip background, border, and text now use theme.palette values
  • Grid lines, axis lines, tick lines, and axis labels use theme-aware colors
  • Active dots on hover are now solid-filled to match the regular dot style

Signed-off-by: Evangelos Skopelitis <eskopelitis@microsoft.com>
@skoeva skoeva self-assigned this Mar 6, 2026
Copilot AI review requested due to automatic review settings March 6, 2026 20:33
@skoeva skoeva added the a11y a11y related issues label Mar 6, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the AKS Desktop plugin’s scaling metrics chart styling to use MUI theme palette values so it renders appropriately in both light and dark modes.

Changes:

  • Replaces hardcoded grid/axis/tick colors with theme.palette tokens.
  • Updates tooltip styling (background, border, text) to be theme-aware.
  • Adjusts active dot styling to use solid fills matching the series color.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Collaborator

@sniok sniok left a comment

Choose a reason for hiding this comment

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

thanks!

@sniok sniok merged commit 7e323b4 into Azure:main Mar 9, 2026
11 checks passed
@skoeva skoeva linked an issue Mar 9, 2026 that may be closed by this pull request
@skoeva skoeva deleted the scaling branch March 9, 2026 21:09
@illume illume mentioned this pull request Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a11y a11y related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] [a11y] scalingtab tooltip text is not readable in dark mode

3 participants