Fix #166

Author: zetachang

CHANGELOG.md

@@ -26,6 +26,7 @@ Whitespace conventions:
 
 ### Fixed
 
+- `$window#on` in `opal-jquery` is broken. (#166)
 - `Element#render` trigger unnecessary re-mounts when called multiple times. (#170)
 - Gets rid of react warnings about updating state during render (#155)
 - Multiple HAML classes (i.e. div.foo.bar) was not working (regression introduced in 0.8.8)

lib/react/top_level.rb

@@ -110,7 +110,7 @@ def self.find(selector)
       selector.dom_node
     rescue
       selector
-    end if selector.respond_to? :dom_node
+    end if `#{selector}.$dom_node !== undefined`
     `$(#{selector})`
   end
 
@@ -126,7 +126,7 @@ def self.[](selector)
     klass.class_eval do
       render(container, params, &block)
     end
-    
+
     React.render(React.create_element(`#{self.to_n}._reactrb_component_class`), self)
   end
 end if Object.const_defined?('Element')

spec/react/opal_jquery_extensions_spec.rb

@@ -56,5 +56,11 @@ def render
 
     expect(Element[renderToDocument(Foo)].html).to eq('hello')
   end
+
+  it "accepts plain js object as selector" do
+    expect {
+      Element[`window`]
+    }.not_to raise_error
+  end
 end
 end