-
-
Notifications
You must be signed in to change notification settings - Fork 760
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
Add a Window Timeout Field in the Command Editor, for Commands Marked as Opening a Window #1829
Comments
I would like that very much! Very clean fork btw! |
Thanks for the kind words - I'll drop in a PR now (although the Chinese translation may need some work) |
@AlexGarrity - No worries there, a couple of Chinese users are contributors here, so if it's rill bad, they can just fix it. |
There are a couple of bits in the PR template regarding changes to documentation and testing - should I sort those out before putting the PR in? |
Nah! I'd need to create documentation for you to edit first 😅 . Your code looks good, if the CI passes I'm happy. I'll work on cutting a release later tonight, although it might take me an extra day or two. I finally realized I wasn't building for M1 Macs, so the build pipeline is kinda broken while I get universal working on Mac OS. |
Alright, no worries then - PR submitted. I remember from my limited experience of doing Mac OS builds that they can be a right pain, so best of luck with it. |
This issue has been automatically locked since there has not been any recent activity since it was closed. Please open a new issue for related bugs. |
🚀 Feature Proposal
When a command is marked as opening a window, a text field should be available to specify the timeout for opening the window.
I have a sample implementation of the feature on a fork of the repo, available here
Motivation
I use Selenium IDE to test a website which occasionally has quite long page load times. As a consequence of this, the default 2 second timeout on opening a window is too short, and frequently causes tests to fail when they're actually fine. It's possible to change this behaviour by editing the
.side
file itself, but it's not exactly the cleanest way of doing it.Example
You have a page that's going to take at least 2 seconds to load due to server-side rendering, and Selenium IDE is failing the test because it's taking too long. You change the timeout to a longer duration, using the window timeout field, and now it passes as expected - without having to manually tweak the
.side
file.The text was updated successfully, but these errors were encountered: