Aakansha Doshi
Aakansha Doshi
@sumit0001jnv we have updated the API and introduced a new component [LiveCollaborationTrigger](https://github.com/excalidraw/excalidraw/blob/master/src/packages/excalidraw/README.md#livecollaborationtrigger) which you will need to use to add the collab component. You can try out the `@excalidraw/excalidraw@next` version...
closing this as the package is out - you can upgrade to `0.14.1` version of [@excalidraw/excalidraw](https://www.npmjs.com/package/@excalidraw/excalidraw)
@chunza2542 does it work fine when you don't add the asset path ? (It should try to get the path from `unpkg` in that case)
@ignacio-cuadra how about targetting one thing at a time ? so Only `text-wrap` refactor in this PR so its easy to review and test as well.
> @ad1992 the new wrapText dont remove the trailing spaces, the spaces are controlled at draw the text. Because of this, it is necessary to rewrite code on renderElements file....
> I updated the tests. considering that each character is equal to 10 pixels, and the container has a padding of 5px on each side, the previous tests were wrong....
@ignacio-cuadra thanks for the PR! Doing quick testing on the PR and here are some issues I came across Try editing text / resize you will see some displacement of...
> @ad1992 > > Hi, i have been working on this and my conclusion is that it's necessary to use Intl.Segmenter to fix emojis problem (and other issues related to...
@ignacio-cuadra once I merge https://github.com/excalidraw/excalidraw/pull/6187, the tests might need to get updated in your PR as now we will be moving to `canvas` measurements completely so mocking `measureText` for DOM...
@ignacio-cuadra I was reviewing the code, can we split this into two PRs? 1. With only Refactoring for better readability (should work the same as prod) 2. With the bug...