Charusso added a comment. In D65250#1601187 <https://reviews.llvm.org/D65250#1601187>, @NoQ wrote:
> In D65250#1600776 <https://reviews.llvm.org/D65250#1600776>, @grandinj wrote: > > > There is no need to wrap SVG in HTML if you want to display it in a > > web-browser > > > I guess it's worth it when the recipient doesn't trust html attachments (eg., > for sending to the mailing lists). You could put HTML inside the SVG (https://developer.mozilla.org/en-US/docs/Web/SVG/Element/foreignObject). That is a cool feature just I have not managed to use it yet, but I would use it for better performance in loading the SVG, and that definitely needs an HTML wrapper. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65250/new/ https://reviews.llvm.org/D65250 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
