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
Canvas: Add icon value mapping #59013
Conversation
Drone build failed: https://drone.grafana.net/grafana/grafana-enterprise/42772 |
Drone build failed: https://drone.grafana.net/grafana/grafana-enterprise/42947 |
Drone build failed: https://drone.grafana.net/grafana/grafana-enterprise/42968 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, tested locally and behaves as expected! Great job!
(cherry picked from commit e157ef1)
Canvas: Add icon value mapping (grafana#59013) (cherry picked from commit e157ef1) Co-authored-by: Nathan Marrs <nathanielmarrs@gmail.com>
Received some feedback from user that #58492 did not resolve their use case. Thinking on this more it seems that a user's datasource would not return icon paths. Instead the most likely case is that a user would use value mappings to tie a given numeric value to a given icon.
We added support for icon value mapping a while ago in #44503 but limited it to only the alpha icon panel. This PR includes icon value mapping support into the canvas panel.
See community comment here
Also confirmed user can use both color and icon value mapping together
canvas.icon.value.mapping.demo.mov
Which issue(s) does this PR fix?:
Fixes #59011