> @@ -2,6 +2,16 @@
>  require 'forwardable'
>  
>  module Watir
> +  
> +  #
> +  # Configurable default wait time for wait methods.
> +  #
> +  
> +  @default_timeout = 30  
> +  class << self
> +    attr_accessor :default_timeout    

I'm not sure about this, since we already have some options on the `Watir` 
module, like `Watir.always_locate` and `Watir.prefer_css`. Doesn't it make 
sense to keep them in one place? A regular user doesn't necessarily need to 
know that the `Watir::Wait` module exists. If so the `@default_timeout` should 
be moved 
[here](https://github.com/watir/watir-webdriver/blob/4e0d343315c8324b938f423676531021030090ff/lib/watir-webdriver.rb#L24-L26).

---
Reply to this email directly or view it on GitHub:
https://github.com/watir/watir-webdriver/pull/230/files#r7854202
_______________________________________________
Wtr-development mailing list
[email protected]
http://rubyforge.org/mailman/listinfo/wtr-development

Reply via email to