Support setting html content through Element#html()

opal · Nov 28, 2013 · 374e467 · elia · Dec 1, 2013 · adambeynon
1 parent dfdeedc
commit 374e467
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 2 deletions.
diff --git a/opal/opal-jquery/element.rb b/opal/opal-jquery/element.rb
@@ -211,8 +211,14 @@ def first
     `self.length ? self.first() : nil`
   end
 
-  def html
-    `self.html() || ""`
+  def html(content = undefined)
+    %x{
+      if (content != null) {
+        return self.html(content);
+      }
+
+      return self.html() || '';
+    }
   end
 
   def id

diff --git a/spec/element_spec.rb b/spec/element_spec.rb
@@ -184,3 +184,21 @@
     end
   end
 end
+
+describe "Element#html" do
+  html <<-HTML
+    <div id="foo">bar</div>
+  HTML
+
+  it "retrieves the inner html content for the element" do
+    expect(Element.id('foo').html).to include('bar')
+  end
+
+  it "can be used to set inner html of element by passing string" do
+    foo = Element.id 'foo'
+    foo.html "different content"
+
+    expect(foo.html).to_not include('bar')
+    expect(foo.html).to include('different content')
+  end
+end