[GitHub] [superset] rusackas edited a comment on pull request #13111: feat: add Echarts Graph chart

[GitBox <gi...@apache.org>]

rusackas edited a comment on pull request #13111:
URL: https://github.com/apache/superset/pull/13111#issuecomment-783631477


   Agreed on merging this PR. I have some thoughts about the tooltip (and related work) but have been pocketing the ideas for a later date. Sneak preview of things I'd like to bring up soon-ish:
   • Displaying the tooltip, and having it defined by the plugin as part of a metadata object
   • Adding MORE (pre-defined) metadata fields to viz plugins (i.e. tags (realtime support, library used, annotations supported, crossfilters supported, etc)
   • Adding the viz category, per SIP-34 or something like it (design needed)
   • Adding collapsibles and tag filters to the viz gallery to take advantage of the above metadata
   ... but all that can wait :)


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