Nemanja Tosic
Nemanja Tosic
I have tested both versions provided and copy/paste does not work on either 0.61.3 nor 0.72.6. I would also suggest we tweak the wording of the issue a bit (of...
I don't really like using both mocha and jest like it is now, and this PR adds more jest code. I think one test runner (used loosely as there's more...
`at` does default to selection. The issue does go away when you remove `at` from setNodes in the bold handler though, and i believe the issue is that when `at`...
[You can import CommonJS from ESM](https://nodejs.org/api/esm.html#esm_interoperability_with_commonjs): > An import statement can reference an ES module or a CommonJS module. (automerge-backend-wasm being the CommonJS module, and the test package an ESM,...
We currently do not use automerge, so i can't verify.
I use a setup where react-use is imported by both a node module (node 16) and nextjs - node cannot use named exports reliably for CJS and if i use...
In general I've a couple of things i would ideally like to see with this PR, which i did in the following commits that we could cherry-pick if they look...
https://github.com/ianstormtaylor/slate/commit/e2b8294cf7a4d36349f381d3ac1f345c4b67fd0c is not focused on performance improvements, rather on getting decorations to with the stock [useSyncExternalStore](https://reactjs.org/docs/hooks-reference.html#usesyncexternalstore), which does not have the memoization parameters like the shim. I'm not sure if...
What is the expected behavior for `text.insertAt(0, 'abc')`? One test specifying "should support inserting a single element" is asserting that the text equals abc: https://github.com/automerge/automerge/blob/performance/test/typescript_test.ts#L327-L331 This test is in contradiction...
I'm not a maintainer so i wouldn't know. I'm pointing out how the tests prior to the PR were handling the insertion of strings and it appears to me that...