You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2022/04/21 07:05:50 UTC

[GitHub] [superset] villebro commented on pull request #19794: feat(explore): Rearrange controls in most popular charts

villebro commented on PR #19794:
URL: https://github.com/apache/superset/pull/19794#issuecomment-1104797069

   A few thoughts:
   - I've probably raised this many times before, but I'll do it once more, just in case 😆  The title "Chart" (instead of "Query") could be seen as slightly ambiguous, as everything should affect the chart. While I know that "Query" isn't very business user friendly, maybe there's some other term out there that could convey more clearly what these controls are.
   - I noticed that Mixed Timeseries isn't included in this PR. Could we update that one, too?


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

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


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