Skip to content

Track Responsive-Design carousel resize events#579

Open
issackjohn wants to merge 1 commit into
WebKit:mainfrom
issackjohn:track-carousel-resize-events
Open

Track Responsive-Design carousel resize events#579
issackjohn wants to merge 1 commit into
WebKit:mainfrom
issackjohn:track-carousel-resize-events

Conversation

@issackjohn

Copy link
Copy Markdown
Contributor

Follow-up to the review feedback on #453, which asked to track the number of resize events so we can verify consistency across browsers.

  • Add PageElement.observeResizeEvents() to count distinct content-box width changes delivered by ResizeObserver. The first delivery seeds the baseline and is not counted, and repeated deliveries of the same width are ignored.
  • Observe the recipe carousel during ReduceWidthIn5Steps and IncreaseWidthIn5Steps, and log the observed count so the numbers can be compared across browsers.
  • Add unittests covering baseline seeding, distinct width changes, and duplicate-width deliveries.

@netlify

netlify Bot commented Jul 24, 2026

Copy link
Copy Markdown

Deploy Preview for webkit-speedometer-preview ready!

Name Link
🔨 Latest commit 6e98693
🔍 Latest deploy log https://app.netlify.com/projects/webkit-speedometer-preview/deploys/6a63f7686dd9220008a05d6b
😎 Deploy Preview https://deploy-preview-579--webkit-speedometer-preview.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 project configuration.

Follow-up to the review feedback on WebKit#453, which asked to track the number
of resize events so we can verify consistency across browsers.

- Add PageElement.observeResizeEvents() to count distinct content-box width
  changes delivered by ResizeObserver. The first delivery seeds the baseline
  and is not counted, and repeated deliveries of the same width are ignored.
- Observe the recipe carousel during ReduceWidthIn5Steps and
  IncreaseWidthIn5Steps, and log the observed count so the numbers can be
  compared across browsers.
- Add unittests covering baseline seeding, distinct width changes, and
  duplicate-width deliveries.
@issackjohn
issackjohn force-pushed the track-carousel-resize-events branch from 226cbbf to 6e98693 Compare July 24, 2026 23:38
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