From ece10e59423dd06fe97df442c43f70e2563b6f8a Mon Sep 17 00:00:00 2001 From: coderaiser Date: Tue, 3 Sep 2013 10:02:35 +0000 Subject: [PATCH] refactor(dom) addClass, removeClass, toggleClass --- lib/client/dom.js | 37 ++++++++++++------------------------- 1 file changed, 12 insertions(+), 25 deletions(-) diff --git a/lib/client/dom.js b/lib/client/dom.js index 49bb9531..024af43a 100644 --- a/lib/client/dom.js +++ b/lib/client/dom.js @@ -206,17 +206,12 @@ var CloudCmd, Util, DOM, CloudFunc; * @param pClass */ this.addClass = function(pElement, pClass) { - var lRet; + var lRet = pElement && pClass; - if (pElement) { - var lClassList = pElement.classList; - lRet = !this.isContainClass(pElement, pClass); - - if ( lRet ) - lClassList.add(pClass); - } + if (lRet) + pElement.classList.add(pClass); - return lRet; + return this; }; /** @@ -225,29 +220,21 @@ var CloudCmd, Util, DOM, CloudFunc; * @param pClass */ this.removeClass = function(pElement, pClass) { - var lRet_b = true, - lClassList = pElement.classList; + var lRet = pElement && pClass; - if (pElement && lClassList) - lClassList.remove(pClass); + if (lRet) + pElement.classList.remove(pClass); - else - lRet_b = false; - - return lRet_b; + return this; }; this.toggleClass = function(pElement, pClass) { - var lRet_b = true, - lClassList = pElement.classList; + var lRet = pElement && pClass; - if (pElement && lClassList) - lClassList.toggle(pClass); + if (lRet) + pElement.classList.toggle(pClass); - else - lRet_b = false; - - return lRet_b; + return this; }; /**