-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Use cypress on application bigger that 4000px wide #5871
Comments
Here is the previous PR that upped the max value: #5189 |
Is there a valid reason for the limits? |
No valid reason, I feel that the max should be set in the config with a default lower limit set in the code but this is outside my ability in this codebase at the moment. |
I think the intention is to prevent users from accidentally setting something unreasonably huge and crashing/freezing the browser. I do not think there is a practical reason outside of that. Instead of adding more config options, I'm in favor of just removing the limitation. In the year 2022, people are probably gonna be using Cypress with 16k res screens, so I think we should just save the time now and remove the restriction, or set it to something that is clearly infeasible (like 100k or 1 million). |
As a workaround, you can suppress this error by using Add the following to your specs or to your support file: beforeEach(() => {
cy.stub(Cypress.utils, "throwErrByPath")
.callThrough() // still throw other types of errors
.withArgs("viewport.dimensions_out_of_range").returns() // suppress invalid value errors
}) |
The code for this is done in cypress-io/cypress#5882, but has yet to be released. |
Created an issue to remove/vastly increase the limits: #5892 |
Released in |
Current behavior:
I have a application which is displayed across multiple screens which is bigger that the max screen size of 400px wide
Desired behavior:
The max
cy.viewport()
width and height increased to8000px
.Steps to reproduce: (app code and test code)
https://github.com/digiink/cypress-test-tiny
Versions
Cypress package version: 3.6.1
Cypress binary version: 3.6.1
Mac OS 10.14
Chrome 78
The text was updated successfully, but these errors were encountered: