-
Notifications
You must be signed in to change notification settings - Fork 146
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Masonry block use inner core/image blocks #2117
Conversation
Can we change the arrows to point up and down instead of left and right? I think it's a prop we can set. |
97fb55f
to
42c2bc3
Compare
42c2bc3
to
e59a113
Compare
Not sure if it's expected behaviour, but when adding multiple images at the same time, the gallery reloads a few times (I think the number of images added) before stabilizing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is monumental work, good job! :)
Whenever you feel ready, you can merge, as the failing tests for OpenTable are not related to your PR :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Impressive work!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job! Code review done. I'll test it IRL now :)
Object.prototype.toString.call( selectedImages ) === | ||
'[object FileList]'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
First time I see something like that. Doesn't mean it's wrong
Pushed up fixes to address the formatting feedback. Also found out that the stacked jest tests were formatting incorrectly causing the results not to be correct. I fixed that here too. |
Description
Masonry Block Refactors.
This is moving the Masonry Gallery away from desandro masonry into a pure CSS solution for layout.
New block architecture reflects that of the Core Gallery block in that this implementation uses Core Image inner blocks.
Backward compatible to WP 5.5 by keeping the V1 block around. We conditionally load the V1 block to keep functionality as well as preserve masonry styles in the case of pre 5.6 WordPress.
AC
This criteria has been modified - Notes below.
ISBAT sort images in Masonry Gallery - Something like ascending, descending by upload date, or random, or append to top or bottom.ISBAT access image replace controls both on image and in block toolbarISB using the newer size controls and deprecated the old controlsScreenshots
Types of changes
Major changes discarding all of our old CSS and JS in favor of a mirrored core implemntation using hooks. This new method matches the new paradigm and allows us to refactor other galleries to this paradigm more easily.
How has this been tested?
WIP
Checklist: