Skip to content

Add mobile playwright testing #81

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Add mobile playwright testing #81

wants to merge 4 commits into from

Conversation

hojberg
Copy link
Contributor

@hojberg hojberg commented May 22, 2025

We want to ensure things look good on mobile. Use targeted mobile selectors in playwright testing as well as screenshots for comparisons.

Copy link

netlify bot commented May 22, 2025

Deploy Preview for unison-share-ui ready!

Name Link
🔨 Latest commit ce0a8ac
🔍 Latest deploy log https://app.netlify.com/projects/unison-share-ui/deploys/6839c846ed5ec600080f7746
😎 Deploy Preview https://deploy-preview-81.staging.share.unison-lang.org
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 55 (🟢 up 5 from production)
Accessibility: 93 (no change from production)
Best Practices: 100 (🟢 up 8 from production)
SEO: 75 (no change from production)
PWA: 80 (no change from production)
View the detailed breakdown and full score reports

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

@hojberg hojberg force-pushed the mobile-playwright branch 2 times, most recently from 38e87d1 to 95f9156 Compare May 23, 2025 13:02
@hojberg hojberg force-pushed the mobile-playwright branch from 95f9156 to 66103ff Compare May 29, 2025 20:46
@hojberg hojberg requested a review from a team as a code owner May 29, 2025 20:46
@hojberg hojberg force-pushed the mobile-playwright branch from 66103ff to 26363ca Compare May 29, 2025 21:23
@hojberg hojberg force-pushed the mobile-playwright branch from c6ff26d to ce0a8ac Compare May 30, 2025 15:01
Copy link

@ChrisPenner ChrisPenner left a comment

Choose a reason for hiding this comment

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

It's cool you can see screenshots right in the diff!

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