Harri Lehtola

Results 9 comments of Harri Lehtola

> this leads to pristine being wrong when your do not define initial values and then add and remove all elements from the array But if you **have** defined an...

Thanks for the info! I agree arrays should work the same way as strings, though I guess that just means I would also like strings to work differently. 😅 (Though...

>> The behaviour you want is not implemented for strings either (why should arrays be treated differently?) > I agree arrays should work the same way as strings [...] Actually,...

It's at least this line: https://github.com/atlassian/react-beautiful-dnd/blob/ae7bcf326928fea3615dfed092dd4261a3976e0d/src/view/droppable/connected-droppable.js#L278 This kind of a patch helped to get rid of it in our project: ```diff diff --git a/dist/react-beautiful-dnd.esm.js b/dist/react-beautiful-dnd.esm.js index ecced698aefb8b8e97a196195ef77955941e0b72..4753f5f9ba6c92969c6fa8bf81eb2fda389b4c72 100644 --- a/dist/react-beautiful-dnd.esm.js...

I've understood this project is being sunsetted and replaced with [Pragmatic drag and drop](https://github.com/atlassian/pragmatic-drag-and-drop) (see #2573), and no further development will be done apart from security fixes. The README says:...

> Hi, I have tried to change the file with the patch-package. Thanks for the suggestion. However, I get this error now: > > Uncaught ReferenceError: Droppable is not defined...

Having to deal with optionality adds unnecessary complexity to me. Previously (with 3.0.2), we could initialize the value to `[]` and only deal with arrays, now (3.1.0) we have to...

This seems even more relevant now that [final-form-arrays v3.1.0](https://github.com/final-form/final-form-arrays/releases/tag/v3.1.0) sets the value to `undefined`: > Change `remove` and `removeBatch` behavior to set array value to undefined when all items have...

Most of the vulnerabilities come from [react-scripts](https://www.npmjs.com/package/react-scripts) which hasn't been updated in 2.5 years. We'll need to [eject](https://create-react-app.dev/docs/available-scripts/#npm-run-eject) to basically get rid of that dependency and start managing the setup...