-
Notifications
You must be signed in to change notification settings - Fork 273
feat: support sort the rich tooltip by metric desc[ID-6] #1445
feat: support sort the rich tooltip by metric desc[ID-6] #1445
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/superset/superset-ui/FYBcHm89Mq4fJDRpwSEyudGAghdK |
Codecov Report
@@ Coverage Diff @@
## master #1445 +/- ##
==========================================
+ Coverage 30.42% 30.43% +0.01%
==========================================
Files 497 497
Lines 10003 10012 +9
Branches 1689 1691 +2
==========================================
+ Hits 3043 3047 +4
- Misses 6714 6719 +5
Partials 246 246
Continue to review full report at Codecov.
|
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.
codes LGTM, please PM sign off. cc: @junlincc
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.
Much better UX! Thanks for the improvement
🏆 Enhancements
This PR adds one
tooltip sort by metric
control to support sorting the rich tooltip by metric desc.not checked
checked