Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Graph display ignores custom colours #760

Closed
drgrumpy opened this issue Mar 23, 2020 · 10 comments
Closed

Graph display ignores custom colours #760

drgrumpy opened this issue Mar 23, 2020 · 10 comments

Comments

@drgrumpy
Copy link

Version 8.3

Custom colours seem to be ignored and replaced with another colour.

Custom colours checkbox is checked and I select various colours for outputs, pid settings etc.
click save, then some of the colours seem to get changed...
I think it only affects outputs.....(i.e. inputs seem to keep their assigned colours)

@drgrumpy
Copy link
Author

This is in the dashboard.

@drgrumpy
Copy link
Author

See screenshot in #759 where the colours of the outputs are not the same as those that I selected.

@kizniche
Copy link
Owner

kizniche commented Mar 23, 2020

I can't replicate this behavior. Can you supply screenshots of both your graph as well as your configuration for that graph?

@drgrumpy
Copy link
Author

This is before saving:
image

@drgrumpy
Copy link
Author

This is after saving the above colour settings...
image
SSR1 should be red, but has taken the colour above....
Lights is red but should be pale yellow
PWM should be blue but is pale yellow..
I am wondering if there is a for loop that is indexed incorrectly ?

@drgrumpy
Copy link
Author

And if I make a change to another value (like changing data grouping) and save again then I get another change to the colours....
image

@kizniche
Copy link
Owner

The issue is the ordering. The colors are correctly being applied to the graph series in the exact same order as in the configuration, however PIDs come before Outputs in the configuration and PIDs come after Outputs in the graph series.

@kizniche
Copy link
Owner

Fixed the order, now working on another issue: displaying the proper color from the database on the configuration page.

@kizniche
Copy link
Owner

Found the last issue and pushed fixes. They'll be available in the next release.

@drgrumpy
Copy link
Author

Great, many thanks.

@kizniche kizniche closed this as completed Apr 3, 2020
kizniche added a commit that referenced this issue Jun 14, 2020
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

No branches or pull requests

2 participants