react-responsive-masonry icon indicating copy to clipboard operation
react-responsive-masonry copied to clipboard

ResponsiveMasonry

Open sarkorbeka opened this issue 1 year ago • 6 comments

image i can't understand, what is the problem? image

sarkorbeka avatar Jul 30 '24 19:07 sarkorbeka

Same here

chintanw3nuts avatar Aug 02 '24 08:08 chintanw3nuts

image

This happens because using defineProperty with only a getter makes it immutable. I think this is an issue with the build system.

I tried to build the code using v21.7.1 and it produces a different output. I am not sure which version the code authors used.

rusith avatar Aug 04 '24 18:08 rusith

I had to build it myself using the latest node version. hope this gets fixed soon (build using a latest node version import from the lib folder)

rusith avatar Aug 04 '24 19:08 rusith

My team had this issue after regenerating the package-lock.json file during updating unrelated dependency

Our workaround is to downgrade to v2.2.0 and make it strict in package.json

terminal:

npm i [email protected]

package.json:

- "react-responsive-masonry": "^2.2.0",
+ "react-responsive-masonry": "2.2.0",

MNITD avatar Sep 10 '24 21:09 MNITD

same issue, v2.2.1 works without issues

AndreyMarchuk avatar Oct 06 '24 09:10 AndreyMarchuk

Regardless of the above, I wonder why a ResponsiveMasonry wrapper component is necessary when the Masonry component itself could be coded to accept a breakpoint object or breakpoint array instead of a number for columnCount (or as another prop) and simply make itself responsive based on it. 😆

ADTC avatar Nov 19 '24 14:11 ADTC