JayHealth commented on PR #36901: URL: https://github.com/apache/superset/pull/36901#issuecomment-3818217289
Hey people, I thought it was good to give this one some space, to see if there's consensus on a way forward. The design element I don't think is addressed the feedback so far, is that pie chart already functions this way, so if a new pop-up menu element is implemented to handle this behaviour, that doesn't cover that the current usage is already not consistent: timeseries splits all columns if drill-by is selected, while pie chart zooms in on a single element. I'm the outsider, so I think I should only give a suggestion, but it is: drill-by means you drill INTO something, so the current behaviour of pie and the behaviour I've implemented is the one you'd expect by drill-by. The current behaviour of time series, belongs in a separate menu item that could be called "split by". That's only my suggestion of course. Perhaps you can come to a consensus from that? -- 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]
