feat(browser): make iframe scalable, improve documentation#6257
feat(browser): make iframe scalable, improve documentation#6257sheremet-va merged 21 commits intovitest-dev:mainfrom
Conversation
|
@userquin I would also love to have https://github.com/user-attachments/assets/29e796ae-6c84-4cf7-8328-b846e3df1f35 in "Running Tests" section - could you help me with that? (need dark mode) |
Fixed just now |
✅ Deploy Preview for vitest-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
So, now the only way to access the browser mode page is from the navbar, maybe we can add the link in the left aside. On mobile the browser mode link will be on top right menu and missing from the left menu hamburger button. (just add the |
My logic was that it is still displayed in the right hamburger on mobile. Isn't it enough? |
|
It is fine, no other entries in the navbar present. FYI: maybe we can remove mastodon, X and discord links from resources navbar menu, already present in the navbar |
33f765a to
8c3f155
Compare
|
@userquin @JessicaSachs made iframe scalable (similarly to how cypress implements it) - please check when you have time 🙏🏻 Note: I didn't change the viewport reset yet, it's a bit tricky (it won't be changed in this PR) |
|
Oof, I forgot about the "Select a test to run" use case 👀 |
from the ui? |
|
The ui doesn't start, Vitest hangs when running |
f9b3593 to
7cd4f1f
Compare
userquin
left a comment
There was a problem hiding this comment.
LGTM
Remember to update images for Vitest ui and coverage (6 in total, 3 per theme): /guide/ui.html and /guide/coverage.html#vitest-ui pages
|
I searched the rationale for the decision to make the iframe scaled, but I didn't find any explanation, except for the comment above saying that "Cypress did it". #6257 (comment) When I'm trying to make visual tests and save snapshots/screenshots, they also appear scaled because of this. |
Description
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:
pnpm-lock.yamlunless you introduce a new test example.Tests
pnpm test:ci.Documentation
pnpm run docscommand.Changesets
feat:,fix:,perf:,docs:, orchore:.