Make WordPress Core

Opened 4 years ago

Closed 3 years ago

Last modified 3 years ago

#48031 closed enhancement (fixed)

Update Masonry

Reported by: desrosj's profile desrosj Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.5 Priority: low
Severity: minor Version:
Component: External Libraries Keywords: has-patch needs-testing has-dev-note
Focuses: javascript Cc:


The masonry-layout package is outdated. Version 3.3.2 is currently packaged. The current version is 4.2.2.

The most notable change when updating this library is that support for IE 8 & 9 would be dropped in Masonry. WordPress Core currently only supports IE 11 and Edge, so this is OK. But, a dev note should probably be published to get the word out in case sites need to support older versions of IE on the front end.

Release Notes:
Version Comparison:

Attachments (3)

48031.patch (1.4 KB) - added by pierlo 4 years ago.
Upgrade Masonry to v4.2.2
48031.2.patch (9.5 KB) - added by pierlo 4 years ago.
Include package-lock.json
48031.3.diff (5.6 KB) - added by Hareesh Pillai 3 years ago.
Patch refreshed

Download all attachments as: .zip

Change History (14)

4 years ago

Upgrade Masonry to v4.2.2

#1 @pierlo
4 years ago

  • Keywords has-patch needs-dev-note added; needs-patch removed

Thanks for the report @desrosj.

This ticket was mentioned in Slack in #core by pierlo. View the logs.

4 years ago

4 years ago

Include package-lock.json

#3 @garrett-eclipse
4 years ago

Hi @pierlo reviewing 48031.2.patch quickly there's alot of potentially unintended changes as I'm seeing several scripts removed that aren't related to masonry;

As well as some others added;

I may be wrong but I believe you want to make sure your patch only updates masonry.

#4 follow-up: @pierlo
4 years ago

Thanks for the review, @garrett-eclipse. Those changes are apart of the dependencies for masonry.

Version 0, edited 4 years ago by pierlo (next)

#5 in reply to: ↑ 4 @garrett-eclipse
4 years ago

  • Keywords needs-testing added

Replying to pierlo:

Thanks for the review, @garrett-eclipse. Those changes are apart of the dependencies for masonry.

masonry-layout@4.2.2, required by WordPress@5.3.0
get-size@2.0.3, required by masonry-layout@4.2.2
outlayer@2.1.1, required by masonry-layout@4.2.2
ev-emitter@1.1.1, required by outlayer@2.1.1
fizzy-ui-utils@2.0.7, required by outlayer@2.1.1
get-size@2.0.3, required by outlayer@2.1.1
desandro-matches-selector@2.0.2, required by fizzy-ui-utils@2.0.7

AHH nice that makes sense, thanks for the explaination @pierlo I'll leave to @desrosj or @SergeyBiryukov to review more thoroughly and test.

#6 @chesio
4 years ago

Please, note that there's already ticket for that: #44780 It has no patch yet though, so I'm not sure which of the two should be marked as duplicate.

#7 @desrosj
4 years ago

#44780 was marked as a duplicate.

@Hareesh Pillai
3 years ago

Patch refreshed

#9 @SergeyBiryukov
3 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 48038:

External Libraries: Update Masonry to 4.2.2 and imagesLoaded to 4.1.4.

Notable changes include dropping support for IE 8 & 9, which are no longer supported by WordPress core either.

Release notes:


Props pierlo, hareesh-pillai, grapplerulrich, pderksen, desrosj, garrett-eclipse, SergeyBiryukov.
Fixes #37675, #48031.

#10 @SergeyBiryukov
3 years ago

  • Milestone changed from Awaiting Review to 5.5

#11 @desrosj
3 years ago

  • Keywords has-dev-note added; needs-dev-note removed
Note: See TracTickets for help on using tickets.