diff options
| author | Igor Minar | 2011-08-14 03:24:09 -0700 |
|---|---|---|
| committer | Igor Minar | 2011-08-15 12:51:41 -0700 |
| commit | 793ecb48176ad6d8866ce38f4242971c81ceda31 (patch) | |
| tree | 0f4a6c975bcab004f7878a4765c56c87b3e34c02 /src/directives.js | |
| parent | 1d45e65f4a026eb0461429f2557a5f0400b7f88e (diff) | |
| download | angular.js-793ecb48176ad6d8866ce38f4242971c81ceda31.tar.bz2 | |
refactor(jqLite): remove jqLite show/hide support
it turns out that even with our tricks, jqLite#show is not usable in
practice and definitely not on par with jQuery. so rather than
introducing half-baked apis which introduce issues, I'm removing them.
I also removed show/hide uses from docs, since they are not needed.
Breaks jqLite.hide/jqLite.show which are no longer available.
Diffstat (limited to 'src/directives.js')
| -rw-r--r-- | src/directives.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/directives.js b/src/directives.js index 54d7feb2..7edbbd6f 100644 --- a/src/directives.js +++ b/src/directives.js @@ -710,7 +710,7 @@ angularDirective("ng:class-even", ngClass(function(i){return i % 2 === 1;})); angularDirective("ng:show", function(expression, element){ return function(element){ this.$watch(expression, function(scope, value){ - toBoolean(value) ? element.show() : element.hide(); + element.css('display', toBoolean(value) ? '' : 'none'); }); }; }); @@ -751,7 +751,7 @@ angularDirective("ng:show", function(expression, element){ angularDirective("ng:hide", function(expression, element){ return function(element){ this.$watch(expression, function(scope, value){ - toBoolean(value) ? element.hide() : element.show(); + element.css('display', toBoolean(value) ? 'none' : ''); }); }; }); |
