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 2021/02/15 19:25:00 UTC

[GitHub] [superset] ktmud commented on pull request #13111: feat: add Echarts Graph chart

ktmud commented on pull request #13111:
URL: https://github.com/apache/superset/pull/13111#issuecomment-779410330


   > > Since the chart type has been renamed, I'm wondering whether we should add a tooltip (or something else) to help old users find the graph chart option:
   > 
   > > 
   > 
   > > <img alt="renamed-chart-type" width="860" src="https://user-images.githubusercontent.com/335541/107873798-1736fb00-6e6a-11eb-94b3-ef4f2893e67a.png">
   > 
   > > 
   > 
   > > But I'm sure people will eventually find it even without this tooltip.
   > 
   > 
   > 
   > Cool,how to do it?
   > 
   > Migration to put notification some table?
   > 
   > @ktmud 
   
   The tooltip is just a suggestion. We can hard-code a closable one-time tooltip if we want to implement it (and not showing it once users have viewed it. You can remember the status via cookie or localStorage).
   
   But as @junlincc mentioned, it may be necessary to do some more aggressive redesign to the viz type selector modal. We should wait for design feedbacks and see if there are better ways to convey this information.


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

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