dom/element/attributes: some fixes for name polyfills

meh · meh · commit 1e07e0f2d234 · 2014-02-04T01:25:53.000+01:00
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';
