justinpark commented on PR #27717:
URL: https://github.com/apache/superset/pull/27717#issuecomment-2035327513

   > I would only keep the background white to not mix it with the draggable 
items and because this looks like a secondary information that does not need to 
be highlighted frequently. Once users view this for the first time, they will 
get used to it.
   
   Sounds good. I'll update accordingly.
   
   > We won't have the Undo link right? I'm assuming that you can't undo the 
applicability of columns because they depend on what's already configured in 
the controls.
   
   I agree with your opinion. Let me remove "undo" now.


-- 
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-unsubscr...@superset.apache.org

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


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

Reply via email to