[GitHub] [superset] graceguo-supercat opened a new issue #14807: [native filter] filter indicator showing up when no filters applicable

Tue, 25 May 2021 01:28:04 -0700 


graceguo-supercat opened a new issue #14807:
URL: https://github.com/apache/superset/issues/14807


   I created a filter_box, and I set it's not applicable to any charts in the 
dashboard. So there is **no filter indicator** showing up for the chart. This 
is correct behavior:
   <img width="1149" alt="Screen Shot 2021-05-25 at 1 17 36 AM" 
src="https://user-images.githubusercontent.com/27990562/119464665-d156c200-bcf7-11eb-91a9-d746ce576639.png";>
   
   But for native filter component, even i set filter scope is not applicable 
for any charts, the chart still show an empty filter indicator:
   <img width="1150" alt="Screen Shot 2021-05-25 at 1 19 29 AM" 
src="https://user-images.githubusercontent.com/27990562/119464912-0fec7c80-bcf8-11eb-8b75-4d2f3f255a7f.png";>
   
   This behavior is wrong. **Empty filter indicator** means the filter field is 
applicable to this chart, but currently there is **_no value applied_**. 
   
   
   ### Environment
   
   latest master.
   
   ### Checklist
   
   Make sure to follow these steps before submitting your issue - thank you!
   
   - [x] I have checked the superset logs for python stacktraces and included 
it here as text if there are any.
   - [x] I have reproduced the issue with at least the latest released version 
of superset.
   - [x] I have checked the issue tracker for the same issue and I haven't 
found one similar.
   
   ### Additional context
   
   cc @junlincc 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to