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
Make NULL value grouping keys show up properly in rich tooltip #6003
Conversation
Codecov Report
@@ Coverage Diff @@
## master #6003 +/- ##
======================================
Coverage 73.5% 73.5%
======================================
Files 72 72
Lines 10001 10001
======================================
Hits 7351 7351
Misses 2650 2650 Continue to review full report at Codecov.
|
0b60d1e
to
dad3e89
Compare
9b241a7
to
c730500
Compare
all relevant tests pass. master is failing, which is causing mine to fail |
@williaster @betodealmeida @graceguo-supercat Can we please get a commit for this small change from Twitter? :-) |
@mistercrunch can we please get a commit on this small change? from twitter |
0f0b2b4
to
0d6d457
Compare
@williaster I made the requested changes, please take a look |
When there was a NULL value grouping key, it would show up in the legend as but would show in the rich tooltip as an empty string. I added an if statement so that the NULL value will show up as in the rich tooltip, as it appears in the legend.
Before:
After: