rusackas commented on issue #35158:
URL: https://github.com/apache/superset/issues/35158#issuecomment-3340075991

   I'm late to the party, but I have some questions before I cast my vote :) 
   
   1. In the "Chart Customization" modal, do we expect to carry over other 
things BESIDE the "groupby" options?
   2. Does the term "Chart Customization" sound like it's customizing the 
chart? Might be confusing - we already have the Chart Properties modal for that.
   3. Is there any reason not to change "Add or edit filters" to be "Manage 
filters and groupings" and just add this into the filter modal (to be renamed) 
alongside scoping as a new tab OR just let it live in the list of other 
filters? It kind of makes sense to me to have filters and group-bys managed in 
the same spot, especially when the UI is strikingly similar.
   4. If this IS going to be a modal of its own, Just wondering what the code 
footprint is here... hopefully it can be pretty DRY/minimal and recycle 
existing stuff.


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