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

Issue with Chart Node #1210

Closed
j0sh162 opened this issue Aug 14, 2024 · 3 comments · Fixed by #1220
Closed

Issue with Chart Node #1210

j0sh162 opened this issue Aug 14, 2024 · 3 comments · Fixed by #1220
Assignees
Labels
bug Something isn't working size:S - 2 Sizing estimation point

Comments

@j0sh162
Copy link

j0sh162 commented Aug 14, 2024

Current Behavior

When working with the chart node and using a bar type and you have more than 9 items it ends up creating the new bars with a very transparent and lightly coloured bar instead of repeating the colours as you can see here.
image

Expected Behavior

I expected the node to reuse the colours which had already been used similar to the pie chart.

Steps To Reproduce

You can reproduce this by dynamically inserting more than 9 items into the chart node.

Environment

  • FlowFuse version: 2.7.0
  • Node.js version: 18.20.4
  • npm version:
  • Platform/OS:
  • Browser: Chrome

Have you provided an initial effort estimate for this issue?

I can not provide an initial effort estimate

@j0sh162 j0sh162 added bug Something isn't working needs-triage Needs looking at to decide what to do labels Aug 14, 2024
@hardillb hardillb transferred this issue from FlowFuse/flowfuse Aug 14, 2024
@hardillb
Copy link
Contributor

I've moved this to the Dashboard repo

@joepavitt joepavitt added size:S - 2 Sizing estimation point and removed needs-triage Needs looking at to decide what to do labels Aug 14, 2024
@joepavitt joepavitt moved this from Backlog to Up Next in Dashboard Backlog Aug 14, 2024
@joepavitt joepavitt moved this from Up Next to In Progress in Dashboard Backlog Aug 20, 2024
@joepavitt joepavitt moved this to In Progress in 🛠 Development Aug 20, 2024
@joepavitt
Copy link
Collaborator

Any updates on this @gayanSandamal?

@gayanSandamal
Copy link
Contributor

Any updates on this @gayanSandamal?

It's almost completed but I didn't create the PR because I need to do more manual testings on this.
I will create the PR and keep it as a draft until finish with the testings

Screenshot 2024-08-20 at 23 33 14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working size:S - 2 Sizing estimation point
Projects
Status: Done
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants