David Luzar

Hey @LeGorge, sorry for the late review. This is awesome! ❤️ One thing: could we adapt it to take zoom into account? The issue is still present if you zoom...

@LeGorge strangely I can't reproduce anymore. I'll review the code tomorrow!

One thing, if you paste an ellipse that was using the old wrapping algo, it requires 2x edits to wrap right:  ``` {"type":"excalidraw/clipboard","elements":[{"id":"NHdwMPZKgWJPtNn9DZGRH","type":"ellipse","x":-3857.193168618643,"y":-4795.797455080002,"width":234,"height":230.3864066699789,"angle":0,"strokeColor":"#5c940d","backgroundColor":"#fab00541","fillStyle":"solid","strokeWidth":2,"strokeStyle":"solid","roughness":1,"opacity":100,"groupIds":[],"roundness":null,"seed":826851654,"version":14,"versionNonce":145058630,"isDeleted":false,"boundElements":[{"type":"text","id":"WUz0xm9C4THfmw5U6q1vH"}],"updated":1676897139026,"link":null,"locked":false},{"id":"WUz0xm9C4THfmw5U6q1vH","type":"text","x":-3851.9999652836536,"y":-4770.604251745012,"width":220,"height":180,"angle":0,"strokeColor":"#5c940d","backgroundColor":"#fab00541","fillStyle":"solid","strokeWidth":2,"strokeStyle":"solid","roughness":1,"opacity":100,"groupIds":[],"roundness":null,"seed":1668500954,"version":218,"versionNonce":1284534362,"isDeleted":false,"boundElements":null,"updated":1676897143111,"link":null,"locked":false,"text":"asdkajsdk jasdkj askjdkaj \nsdkasjd kasdj kadjkajksdj \nkasjdk...

Yeah, it's because we invert the colors using CSS, and then individually apply a counter filter on image elements, but it doesn't undo the initial inversion fully. We could apply...

@anatoliibakhtin do you mean when the text is overlaid over an image, it should not be inverted? Not super easy to do for a general case.

Yea, I agree. We should probably allow groups to be bindable irrespective of what shapes they're comprised of.

@7flash for export, not necessarily, but we'll need to properly decouple the SVG renderer from the canvas. For editing purposes, Excalidraw currently doesn't support SVG-only renderer, although it's also a...

Does zen mode (right-click on canvas -> zen mode) work here?

under consideration

hey @Brikaa — is this actually an issue? You're manually changing a data type that's meant to be a string into a number.