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
Add thumbnail images for Playlab projects #14578
Conversation
apps/src/studio/studioThumbnail.js
Outdated
.then(canvasFromImage) | ||
.then(createThumbnail) | ||
.then(canvasToBlob) | ||
.then(project.saveThumbnail); |
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.
Would it make sense to move more of this logic into imageUtils? i.e. have something like
captureSvgScreenshot(document.getElementById('svgStudio'), MIN_CAPTURE_INTERVAL_MS);
I suspect we will have other svg apps that could then reuse much of this?
@@ -103,7 +104,6 @@ stage 'Pixelation' | |||
level 'Pixelation - Lesson 14 - Make the Letter A' | |||
level 'Pixelation - Lesson 15 - Complete 3-bit color' | |||
level 'Pixelation - Lesson 15 - Color Shades 4x4' | |||
level 'Pixelation - AllTheThings 4' |
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.
intentionally removing pixelation level?
Good suggestion Brent. PTAL. There is more cleanup I can do affecting other app types, but it probably warrants a follow-on PR. |
lgtm |
Also adds a new project-backed playlab level, so that we can remix it to get a new project in the projects list for use in eyes tests.
The SVG capture process seems a little more CPU intensive than artist or gamelab, so only allow it once per minute.
expected new eyes baselines: https://eyes.applitools.com/app/sessions/00000251909492557021/00000251909492556129/steps/1?&parent=vos3oAR&accountId=mG4DoRfv0ntEgI3XIYBulrKxlZqrHtGwbP3V103ojvLFU110