Skip to content

[Feature] Sync container colors on all the charts#1351

Open
svenvg93 wants to merge 41 commits intohenrygd:mainfrom
svenvg93:366-docker-colors
Open

[Feature] Sync container colors on all the charts#1351
svenvg93 wants to merge 41 commits intohenrygd:mainfrom
svenvg93:366-docker-colors

Conversation

@svenvg93
Copy link
Collaborator

@svenvg93 svenvg93 commented Oct 30, 2025

📃 Description

This PR syncs the colors for the docker containers on all the charts. This way the containers have the same colors in all the charts

🪵 Changelog

➕ Added

@svenvg93
Copy link
Collaborator Author

@henrygd This one is ready to go

@svenvg93
Copy link
Collaborator Author

@henrygd (sorry for the spam) any update on this

@henrygd
Copy link
Owner

henrygd commented Nov 30, 2025

Looks good, but I think it would be best to make this change at the same time we split up the system detail into separate pages or tabs.

Then we can also add a single legend (or include the colors in the table) so it's clearer that the colors are tied to specific containers.

@svenvg93
Copy link
Collaborator Author

svenvg93 commented Dec 1, 2025

With pages/tabs do you mean like there was in the original PR #928 or did you had something else in mind?

@henrygd
Copy link
Owner

henrygd commented Dec 2, 2025

Yep, I think we can split it into different categories:

  • Core metrics
  • All Disks
  • S.M.A.R.T.
  • Containers
  • Services (if Linux)

At least this is the general idea.

@svenvg93
Copy link
Collaborator Author

svenvg93 commented Dec 2, 2025

If you want I can add the container tab to this PR.

@svenvg93 svenvg93 requested a review from henrygd as a code owner January 22, 2026 04:47
@svenvg93
Copy link
Collaborator Author

svenvg93 commented Feb 1, 2026

@henrygd Would it be possible to get this one into the next release? There are some other small improvements open I would like to pick up for you, but this one is blocking me now 😅 .

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.

[Feature]: Keep color for docker container across graphs [Feature]: Docker Charts CPU/Memory Usage should show same color per Container

2 participants