Skip to content

Conversation

@pieperm
Copy link
Collaborator

@pieperm pieperm commented Apr 16, 2024

To test:

  • Log in as an admin
  • Go to REPORTS > Team Skills
  • Select some members who have skills
  • Select some skills, then run search
  • Radar chart should display, and colors should not change unless you refresh the page
  • Note: The radar chart seems to work with only 1 skill, but it is not easy to read without the tooltip

This finishes up the work started by @Richardson-e. Adjustments:

  • Removed nivo dependency
  • Fixed how colors are generated for the radar chart
  • Radar chart now displays tooltip with info about the distribution

There are probably additional enhancements we could make for generating the radar chart colors:

  • Make sure all generated colors are distinct in hue
  • Make sure all generated colors are legible (e.g. yellow, light pastels)
  • Allow the user to regenerate all colors or a color of their choosing

Closes #2184

@mkimberlin mkimberlin merged commit d89b893 into develop Apr 18, 2024
@mkimberlin mkimberlin deleted the feature-2148/replace-nivo-with-recharts-radar-chart-teamskills branch April 18, 2024 19:39
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.

Convert from Nivo to Recharts

5 participants