From: senator <lebbeous@esilibrary.com>
Date: Thu, 3 Feb 2011 20:40:34 +0000 (-0500)
Subject: a thought about the add/remove css funcs
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=b3509c18684c3c97efb83a7e28789031eccf94fe;p=evergreen%2Fequinox.git

a thought about the add/remove css funcs
---

diff --git a/Open-ILS/web/opac/skin/kcls-wire/js/misc.js b/Open-ILS/web/opac/skin/kcls-wire/js/misc.js
index 31165eda49..8ca55fd0c7 100644
--- a/Open-ILS/web/opac/skin/kcls-wire/js/misc.js
+++ b/Open-ILS/web/opac/skin/kcls-wire/js/misc.js
@@ -28,6 +28,10 @@ function swapCSSClass(obj, old, newc) {
 }
 
 function addCSSClass(e,c) {
+    /* XXX I've seen much simpler implementation of this idea that just
+     * do a regexp replace on e.className.  Any reason why we're making
+     * it so hard here? I could see the justification if there's a certain
+     * browser that doesn't cooperate. */
 	if(!e || !c) return;
 
 	var css_class_string = e.className;