[GitHub] [superset] amitmiran137 commented on pull request #17505: feat: Move Apply and ClearAll Button below filters and change clearAll Button to link

[GitBox <gi...@apache.org>]

amitmiran137 commented on pull request #17505:
URL: https://github.com/apache/superset/pull/17505#issuecomment-977593197


   @junlincc I totally agree with the need of keeping the buttons sticky in case of many filters 
   But in cases when you have only a few filters it make sense to stick it to the bottom as part of a regular user flow ( working tip to bottom )
   
   Would you consider that as a configurable feature (either bottom/top ) ? 
   There would be a native filters configuration and by default it would be on top
   In that approach we can make superset flexible to different types of products
   In cases of client facing applications there would be a very summerized dashboard with just a few filters but currently superset is more BI/Operational oriented and that make us struggle of making it intuitive to certain clients 
   
   cc:@jayakrishnankarolilnlsn 


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org