Changeset 6723

Timestamp:

Feb 12, 2015, 9:00:47 AM (8 years ago)

Author:

Nicklas Nordborg

Message:

Fixes #1920: Use 'classList' for manipulating DOM element classes

Location:

trunk/www/include/scripts

Files:

• main-2.js (modified) (5 diffs)
• menu.js (modified) (1 diff)

trunk/www/include/scripts/main-2.js

@@ -430 +430 @@
     if (element)
     {
-      if (element.className.indexOf(className) != -1) return;
-      element.className = element.className+' '+className;
+      element.classList.add(className);
     }
   }
@@ -445 +444 @@
     if (element)
     {
-      element.className = element.className.replace(className, '');
+      element.classList.remove(className);
     }
   }
@@ -465 +464 @@
       if (addIfTrue == undefined)
       {
-        addIfTrue = element.className.indexOf(className) == -1;
-      }
-      if (addIfTrue)
-      {
-        doc.addClass(element, className);
+        element.classList.toggle(className);
+      }
+      else if (addIfTrue)
+      {
+        element.classList.add(className);
       }
       else
       {
-        doc.removeClass(element, className);
+        element.classList.remove(className);
       }
     }
@@ -485 +484 @@
   {
     element = Doc.element(element);
-    return element.className.indexOf('disabled') >= 0;
+    return element.classList.contains('disabled');
   }
   
@@ -751 +750 @@
     if (pointerEvents == null)
     {
-      disabled = event.currentTarget.className.indexOf('disabled')>=0;
+      disabled = event.currentTarget.classList.contains('disabled');
     }
     else

trunk/www/include/scripts/menu.js

@@ -309 +309 @@
   internal.menuOnMouseOver = function(event)
   {
-    if (event.target.className.indexOf('menuitem') == -1) return;
+    if (!event.target.classList.contains('menuitem')) return;
     menu.hideSubMenus(event.currentTarget);
   }