kasiazjc commented on PR #23590: URL: https://github.com/apache/superset/pull/23590#issuecomment-1498999586
> @yousoph @kasiazjc Although the ideas are good, ECharts API currently does not offer this kind of flexibility 😢 . There's no native support for controlling the buttons states or adding a tooltip. I can change the text though, I just didn't because I figure it's a common pattern used by ECharts and people are used to it. One thing I noticed when interacting with the controls for the first time, is that you understand the feature pretty quick, and there's no harm when "trying/testing" the buttons. oooh, ok, that makes sense to me! I think we should keep "inv" if that's the case, because I don't think there is a better name for it and as you said it's widely used :) -- 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]
