kicaak commented on pull request #15433:
URL: https://github.com/apache/superset/pull/15433#issuecomment-886021026


   @michael-s-molina thanks for the feedback! 😌 
   
   > One problem that we need to figure out is about the value that represents 
the hierarchy when collapsed. Currently, it changes depending on which levels 
are already filled and this brings the confusion that you mentioned.
   
   Regarding this one - I think that parent filter should always be displayed 
when the hierarchy is collapsed, because otherwise it is hard to follow what 
happens. We can add a label or other indicator so the user knows that there are 
fields that are empty. cc: @junlincc what do you think? 
   
   > I think we need a more clear distinction of the hierarchy group when it's 
expanded. Maybe a blue background color or a highlight border or other fancy 
way.
   
   Agreed, for now I added a divider, but I think it won't work in the long 
run. The only reasonable solution here is background colour when hierarchy is 
expanded, but I would not like to use the blueish preset's colour (even if 
ligter version) as this one should be for special actions/components. Grey may 
blend with the dashboard, but I will see what else I can do. Also additional 
thing would be setting up a rule that only one group can be expanded at a time 
which would help I think. 


-- 
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: [email protected]

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