Notes on using webpack in the readme about react-masonry-component HOT 7 CLOSED

Great thanks all.

I have added a deprecation notice and will remove this section of the README in due time.

Closing this issue now, please feel free to open if needed :-)

from react-masonry-component.

Maybe this was necessary with older versions of masonry and imagesloaded ??

from react-masonry-component.

Hi @hburrows

Which version of Webpack are you using?

At the time of writing the Readme, the import order Webpack used was AMD then commonjs. This meant that some dependencies didn't play nice (imagesloaded plus others)

Things may have changed though, so thanks for bringing this up :-)

from react-masonry-component.

@afram Thanks for the followup. I'm currently using Webpack version 1.13.2 (which is the most recent in the 1.X line).

from react-masonry-component.

Hmm, I wouldn't have thought they would not change such a critical implementation detail without a major version bump, so I'd be very surprised if the import order has changed in that version. I'll check my setup when I have a second and report back.

from react-masonry-component.

I also installed the npm package without any changes to my webpack configuration (I don't use the import-loader).
It works for me, I cannot find any noticeable problems.

I did look into all the top level packages used by this component and all seem to support webpack officially. Probably that is what changed.

The only thing which could still behave weird is this package's conditional "require" statements which depend on the "isBrowser" boolean. I am not sure how webpack will handle that but I assume it would still require them which would mean they are eventually bundled in all the builds (server side and client side if you do both).

I am using that in SSR scenario and all seems to work as said before.

webpack: 1.13.3
-- [email protected]
  +-- [email protected]
  | `-- [email protected]
  +-- [email protected]
  | `-- [email protected]
  +-- [email protected]
  +-- [email protected]
  +-- [email protected]
  `-- [email protected]
    +-- [email protected]
    `-- [email protected]
      `-- [email protected]
        `-- [email protected]

from react-masonry-component.

👍 I can confirm that things work fine without imports-loader for masonry-layout 4.x.
Hunting down the original issue and trying to find a reason for why things work now, I stumbled across this comment by the Masonry author:

Masonry v4.0 and fizzy-ui-utils v2.0.1 has a fix for this issue, using the same package names for AMD and CommonJS dependencies.

from react-masonry-component.