[GitHub] [superset] graceguo-supercat edited a comment on pull request #16154: feat(dashboard): Let users re-arrange native filters

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

graceguo-supercat edited a comment on pull request #16154:
URL: https://github.com/apache/superset/pull/16154#issuecomment-909743589


   Thanks @michael-s-molina and @m-ajay I also played a little bit, here is my feedback:
   
   ![Screen_Shot_2021-08-31_at_3_59_15_PM-2](https://user-images.githubusercontent.com/27990562/131589699-f0d3d294-bbe7-4a15-9f63-045513b2b8c0.jpg)
   
   1. only parent (top level filters) are shown in the filters list
   2. In the edit mode, nested filters are also dragable and could be reordered. But there is no use to do this change, since nested filters don't show up in the filters list. In my opinion, we should probably not offer any action that is no use :)
   
   I understand we want to allow user fully control the nested filter so we show them as first-class filter component. But make user move them up and down do not make sense to me. So my suggestion is:
   - make child filter always next to parent filter,
   - when user moves parent filter, children should be moved along,
   - children (nested filters) itself, should not be dragable or rearrangeable.
   
   @m-ajay @michael-s-molina @junlincc please let me know your opinion about this behavior.
   


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