-
Notifications
You must be signed in to change notification settings - Fork 11
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
Cannot read property '0' of null #59
Comments
We ran into this problem when we upgraded the project from vue-cli to vite. |
@renevanderheijden just curious how you tested the PR changes were fine since the PR changes are not published to npm? having the same issues after moving to vite as well........ |
@wingy3181 As I don't expect that this package will be updated anytime soon, I just removed the cypress-plugin-tab package from the project and made a local implementation. I copied index.js to our project, and updated our Cypress Chainable interface with the content of index.d.ts to add TypeScript support. After that our tests were running fine again, and with less external dependencies. |
@wingy3181 We use patch-package to make small changes to dependencies. Or if you use pnpm it has patching built in. |
@renevanderheijden @AndrewLeedham Thanks for your quick replies..... was weighing up the discussions in cypress-io/cypress#299 and whether i should use https://github.com/dmtrKovalenko/cypress-real-events instead....... |
@wingy3181 We did actually move to |
@AndrewLeedham what kind of Cypress 10 compatibility issues? how are you finding it? i just know its limited to chromium browsers and is using native events so its more realistic? we are using cypress 11 and just at the beginning of writing some e2e tests... so too early to tell but would be good to know so i can pre-emptively assess it.... |
@wingy3181 see #61. Nothing bad to say about |
It seems that some recent browser changes have started causing issues in
platform.js
which is a dependency ofa11y.js
used here. More details can be seen on theplatform.js
issue here: bestiejs/platform.js#196. It seemsa11y.js
hasn't been touched for 5 years andplatform.js
for 2. So I doubt they will get patched anytime soon. Given this package only uses 1 method froma11y.js
it seems like we could probably just include that functionality internally, and removed the need for any dependencies?The text was updated successfully, but these errors were encountered: