diff options
| author | Max Martinsson | 2012-06-07 17:25:35 +0200 | 
|---|---|---|
| committer | Misko Hevery | 2012-09-06 15:49:48 -0700 | 
| commit | 79bb7b1f0b6ded0ca84660f387f406de98f5bdf1 (patch) | |
| tree | 6f885f3f225b5145fd4c4f4281df8584c6a60c36 /src/ng/directive | |
| parent | b936e52874fe0173c6d4ba0a84f45deac67518ac (diff) | |
| download | angular.js-79bb7b1f0b6ded0ca84660f387f406de98f5bdf1.tar.bz2 | |
fix(ngClass): works with class interpolation
Closes #1016
Diffstat (limited to 'src/ng/directive')
| -rw-r--r-- | src/ng/directive/ngClass.js | 26 | 
1 files changed, 17 insertions, 9 deletions
diff --git a/src/ng/directive/ngClass.js b/src/ng/directive/ngClass.js index e054d4c6..6ba73e60 100644 --- a/src/ng/directive/ngClass.js +++ b/src/ng/directive/ngClass.js @@ -3,17 +3,25 @@  function classDirective(name, selector) {    name = 'ngClass' + name;    return ngDirective(function(scope, element, attr) { -    scope.$watch(attr[name], function(newVal, oldVal) { +    // Reusable function for re-applying the ngClass +    function reapply(newVal, oldVal) {        if (selector === true || scope.$index % 2 === selector) {          if (oldVal && (newVal !== oldVal)) { -           if (isObject(oldVal) && !isArray(oldVal)) -             oldVal = map(oldVal, function(v, k) { if (v) return k }); -           element.removeClass(isArray(oldVal) ? oldVal.join(' ') : oldVal); -         } -         if (isObject(newVal) && !isArray(newVal)) -            newVal = map(newVal, function(v, k) { if (v) return k }); -         if (newVal) element.addClass(isArray(newVal) ? newVal.join(' ') : newVal);      } -    }, true); +          if (isObject(oldVal) && !isArray(oldVal)) +            oldVal = map(oldVal, function(v, k) { if (v) return k }); +          element.removeClass(isArray(oldVal) ? oldVal.join(' ') : oldVal); +        } +        if (isObject(newVal) && !isArray(newVal)) +          newVal = map(newVal, function(v, k) { if (v) return k }); +        if (newVal) element.addClass(isArray(newVal) ? newVal.join(' ') : newVal); +      } +    }; +    scope.$watch(attr[name], reapply, true); + +    attr.$observe('class', function(value) { +      var ngClass = scope.$eval(attr[name]); +      reapply(ngClass, ngClass); +    });    });  }  | 
