Skip to content

Conversation

@danleh
Copy link
Contributor

@danleh danleh commented Feb 6, 2026

This is based on top of #274, so let's merge that one first. @camillobruni

This PR:

  • Adds descriptions for all Wasm workloads.
  • Extends the test to check that the <dt> tag text contents matches the id="..." id property.
  • Adds a CSS style for non-default workloads (slightly grayed-out).
  • Extends the text to check that all non-default workloads from the JetStreamDriver.js have this CSS class set.
  • (very minor drive-by:) Add some padding at the bottom of in-depth.html

Fixes the wasm part of #199

@netlify
Copy link

netlify bot commented Feb 6, 2026

Deploy Preview for webkit-jetstream-preview ready!

Name Link
🔨 Latest commit e89c4ec
🔍 Latest deploy log https://app.netlify.com/projects/webkit-jetstream-preview/deploys/698c9d0230a2f20008c961be
😎 Deploy Preview https://deploy-preview-275--webkit-jetstream-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.

@danleh danleh mentioned this pull request Feb 6, 2026
@danleh danleh requested a review from camillobruni February 6, 2026 22:18
Copy link
Contributor

@kmiller68 kmiller68 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with some suggestions

@danleh danleh merged commit 6588e89 into WebKit:main Feb 11, 2026
23 checks passed
@danleh danleh deleted the update-in-depth branch February 11, 2026 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants