[GitHub] [echarts] pissang edited a comment on issue #15062: [SVG] Allow adding DOM to the chart in SVG renderer through `foreignObject`

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

pissang edited a comment on issue #15062:
URL: https://github.com/apache/echarts/issues/15062#issuecomment-852007778


   It's a very attractive feature! But I think we should try not to break the consistency between SVG and Canvas renderer as much as possible.  So this should be a feature that canvas also can have.
   
   For API design, since ForeignObject is a concept that only SVG has. It's better to use a more general API like `zrender.HTMLElement`.


-- 
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: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org