Skip to content
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: Add integration test for dynamic heights v2 and masonry testing page #4021

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

JosAnGoCa
Copy link
Contributor

Summary

Create a new page for the integration tests that contains to multi column items with a column span of 3, also created integration test for dynamic heights v2 that use this page

What changed?

A new page has been created to support integration tests, featuring multi-column items configured with a column span of 3.
This page is designed to test not only dynamic height adjustments but also the positioning of multi-column items and the handling of dynamic batch sizing. The access is via the url paramter fixedThreeColItems=true

image

A new test file has been introduced to rigorously test the Dynamic Heights V2 algorithm. These tests cover a variety of use cases, including: Pushing a multi-column item, Double resizing, Partial pushing of a column. This comprehensive testing ensures that the algorithm handles different scenarios effectively and maintains stable performance.

image
image

Why?

The purpose of these changes is to comprehensively verify the Dynamic Heights V2 algorithm's performance and stability in a visually complex environment. By using a page that simulates a variety of layout scenarios and content variations, these tests offer a robust evaluation of algorithmic behavior across numerous use cases.

Checklist

  • Added integration tests
  • Created a new testing page

@JosAnGoCa JosAnGoCa requested a review from a team as a code owner February 28, 2025 22:33
Copy link

netlify bot commented Feb 28, 2025

Deploy Preview for gestalt ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit c0ff3c2
🔍 Latest deploy log https://app.netlify.com/sites/gestalt/deploys/67d076b5a27abb0008043602
😎 Deploy Preview https://deploy-preview-4021--gestalt.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@JosAnGoCa JosAnGoCa changed the title Masonry : add integration test for dynamic heights v2 and page with two tree column items Masonry : add integration test for dynamic heights v2 and masonry testing page Feb 28, 2025
@JosAnGoCa JosAnGoCa changed the title Masonry : add integration test for dynamic heights v2 and masonry testing page Masonry : Add integration test for dynamic heights v2 and masonry testing page Feb 28, 2025
@JosAnGoCa JosAnGoCa changed the title Masonry : Add integration test for dynamic heights v2 and masonry testing page Masonry: Add integration test for dynamic heights v2 and masonry testing page Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant