dom/element/attributes: some fixes for name polyfills

opal · Feb 4, 2014 · 1e07e0f · dlee · Feb 4, 2014 · meh
1 parent 0dfd47b
commit 1e07e0f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/opal/browser/dom/element/attributes.rb b/opal/browser/dom/element/attributes.rb
@@ -1,14 +1,14 @@
 module Browser; module DOM; class Element < Node
 
 class Attributes
-  @@normalize = {}
+  @@normalize = `{}`
 
   if Browser.supports? 'Element.className'
-    @@normalize[:class] = :className
+    `#@@normalize['class'] = 'className'`
   end
 
   if Browser.supports? 'Element.htmlFor'
-    @@normalize[:for] = :htmlFor
+    `#@@normalize['for'] = 'htmlFor'`
   end
 
   attr_reader :namespace
@@ -20,7 +20,7 @@ def initialize(element, options)
   end
 
   def [](name, options = {})
-    name = @@normalize[name] || name
+    name = `#@@normalize[name] || name`
 
     if namespace = options[:namespace] || @namespace
       `#@native.getAttributeNS(#{namespace.to_s}, #{name.to_s}) || nil`
@@ -30,7 +30,7 @@ def [](name, options = {})
   end
 
   def []=(name, value, options = {})
-    name = @@normalize[name] || name
+    name = `#@@normalize[name] || name`
 
     if namespace = options[:namespace] || @namespace
       `#@native.setAttributeNS(#{namespace.to_s}, #{name.to_s}, #{value})`

diff --git a/opal/browser/support.rb b/opal/browser/support.rb
@@ -122,15 +122,15 @@ def self.supports?(feature)
 
       when 'Element.for'
         %x{
-          var div = document.createElement("div");
+          var div = document.createElement("label");
               div.setAttribute('for', 'x');
 
           return div.htmlFor === 'x';
         }
 
       when 'Element.htmlFor'
         %x{
-          var div = document.createElement("div");
+          var div = document.createElement("label");
               div.setAttribute('htmlFor', 'x');
 
           return div.htmlFor === 'x';