-
Notifications
You must be signed in to change notification settings - Fork 6
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
Better control over browser's position and size is needed #1
Comments
For info, solution 1 is already implemented: the computed viewport offset is not relative to the screen, but to the browser window, so if the browser window is simply moved in the screen, the positions should still be computed correctly. |
But I think the implementation of 1) is fragile since e.g. Chrome displays a yellow warning bar at the top of the browser, if I close this bar, then the viewport position relative to browser window changes, and robot will malfunction. I think the suggestion 2) is better than command line flag to maximize the viewport (but this flag perhaps also could be added): by manually starting the calibration, the user has the greatest flexibility to do anything he needs with the browser window. The other issue is e.g. with IE displaying a bar at the bottom of the browser asking "do you want IE to be a default browser" etc. which when displayed, makes calibration fail. I think the feature could sit behind a command line flag ( |
Having said that, it was a good try in the first place to automate the calibration, but we didn't know the issues that arise when the thing is automated; right now it seems it's better to avoid automation since it can lead to problems, and just one click needed from the user is not a big issue IMO. |
@jakub-g You can have a look to the new robot-server which does the calibration just before each test which needs it (during the call to getOffset). |
Currently the robot starts the requested browser, maximizes it, and calibrates itself by measuring the viewport offsets of the browser wrt the OS screen.
This is however very suboptimal in certain cases. For instance, I may want to start dev tools and don't want the browser to be full screen for that reason. Also, I may want to resize a virtual machine's window.
A solution could be
Solution 2 seems probably easier to implement and more robust.
The text was updated successfully, but these errors were encountered: