-
-
Notifications
You must be signed in to change notification settings - Fork 125
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
Limit emulation to enabling slowmoable #331
Conversation
@route would you be able to take a look at this please? |
@aaronjensen could you please rebase it with master? I don't see how to run tests for this PR |
ffb3088
to
e9c4d8a
Compare
Rebased. When you say you don't see how to run tests, are you saying that you don't see any tests or you this PR somehow breaks the normal means of running tests? The emulation device metric overrides were not previously tested afaict, so I wasn't sure how to add a test for them. |
I do see a test failure actually in browser_spec.rb, specifically:
|
@aaronjensen I was talking about a button to "Approve and Run tests" for this PR. Now it appeared. We need to investigate this failure. Cannot merge it with a breaking test ;) |
e9c4d8a
to
a3703c4
Compare
Updated to only disable overriding the device scale factor. Setting the height and width still work and are necessary to allow controlling height and width when using full screen. |
Looks good now! Sorry that it took me so long! ❤️ |
Thank you! |
@route sorry to bother you again, but do you have an idea when you plan to release a new gem version with this change? Thank you! |
I don’t know, when it’s time :) use master |
Understood, thanks for the response. |
Fixes #330
If the line cannot be removed (because
slowmoable
needs to stay enabled) then this will disable the override of everything else.