-
Notifications
You must be signed in to change notification settings - Fork 272
feat(core): add support for empty results and refresh indicator #1121
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/superset/superset-ui/GANMb36WyU9MWbiPxgDukbxoxmB5 |
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
7d87b19
to
28a968a
Compare
a6428e0
to
fccb805
Compare
fccb805
to
0b6965e
Compare
Codecov Report
@@ Coverage Diff @@
## master #1121 +/- ##
==========================================
+ Coverage 28.99% 29.02% +0.03%
==========================================
Files 462 462
Lines 9208 9212 +4
Branches 1452 1453 +1
==========================================
+ Hits 2670 2674 +4
Misses 6334 6334
Partials 204 204
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.
Re-review - LGTM again!
🏆 Enhancements
Add support for empty datasets without rendering the "No Results" placeholder text and an flag to notify the chart that it's refreshing data without being forced to use the default spinner. Needed to be able to add support for "Search all filter options" feature in Select Filter.
See below an example of the native select filter with the feature turned on (the chart loading times have been exaggerated by placing a sleep on the backend):
https://user-images.githubusercontent.com/33317356/118638272-5402ed00-b7df-11eb-9a2c-a9e78b297c63.mp4