Skip to content

Add button and shortcut to download results JSON#103

Open
sunnyps wants to merge 1 commit intoWebKit:mainfrom
sunnyps:push-vyupswvntvpt
Open

Add button and shortcut to download results JSON#103
sunnyps wants to merge 1 commit intoWebKit:mainfrom
sunnyps:push-vyupswvntvpt

Conversation

@sunnyps
Copy link
Copy Markdown

@sunnyps sunnyps commented Mar 3, 2026

No description provided.

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 3, 2026

Deploy Preview for webkit-motionmark-preview ready!

Name Link
🔨 Latest commit 80f97b2
🔍 Latest deploy log https://app.netlify.com/projects/webkit-motionmark-preview/deploys/69c31be12ab58a0008880e98
😎 Deploy Preview https://deploy-preview-103--webkit-motionmark-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.

@sunnyps sunnyps force-pushed the push-vyupswvntvpt branch from 1d3f4fe to cdda051 Compare March 3, 2026 23:22
@@ -158,6 +158,7 @@ <h1>MotionMark score</h1>
<button onclick="benchmarkController.restartBenchmark()">Test Again</button>
<p>
'j': Show JSON results<br/>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think we should show small gray text that lists these shortcuts on the end screen.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I'm not sure what you mean here since we already have some text listing the shortcuts:

Screenshot 2026-03-24 at 3 20 55 PM

@sunnyps sunnyps force-pushed the push-vyupswvntvpt branch from cdda051 to 8712602 Compare March 24, 2026 22:19
@sunnyps sunnyps force-pushed the push-vyupswvntvpt branch from 8712602 to 80f97b2 Compare March 24, 2026 23:18
@sunnyps
Copy link
Copy Markdown
Author

sunnyps commented Mar 25, 2026

There should now be a gap between the Done and Download buttons in the main benchmark runner (after you press 'j' to show the debug info).

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.

2 participants