diff options
| author | Keir Mierle | 2013-04-21 16:37:53 -0700 |
|---|---|---|
| committer | Pete Bacon Darwin | 2013-04-22 12:52:43 +0100 |
| commit | de5352cfcfa3705bb2ea9f1ca4ee71a2ce18ceb5 (patch) | |
| tree | 99a43e40f29c5ed54a8fa052a3953cbc3426cb7f /docs/content/guide/compiler.ngdoc | |
| parent | ae26ed994e6df3c178e9c6e1346d6e4e6b7736ad (diff) | |
| download | angular.js-de5352cfcfa3705bb2ea9f1ca4ee71a2ce18ceb5.tar.bz2 | |
docs(compiler): fix variable scope in drag sample
Note that without this fix, if you add a second draggable element, the
two instances clobber each other since there is only one set of
startx/starty/x/y variables.
Here is an example: http://plnkr.co/edit/aGrLXcIo2SuaePuAdfmQ?p=preview.
On the surface it looks like it would be fine because you only have one
mouse but in practice the start position jumps when you start dragging.
Here it is fixed: http://plnkr.co/edit/VuvPasuumtCeiVRisYKQ?p=preview
Diffstat (limited to 'docs/content/guide/compiler.ngdoc')
| -rw-r--r-- | docs/content/guide/compiler.ngdoc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/docs/content/guide/compiler.ngdoc b/docs/content/guide/compiler.ngdoc index 9915107b..d315df45 100644 --- a/docs/content/guide/compiler.ngdoc +++ b/docs/content/guide/compiler.ngdoc @@ -70,8 +70,8 @@ Here is a directive which makes any element draggable. Notice the `draggable` at <file name="script.js"> angular.module('drag', []). directive('draggable', function($document) { - var startX=0, startY=0, x = 0, y = 0; return function(scope, element, attr) { + var startX = 0, startY = 0, x = 0, y = 0; element.css({ position: 'relative', border: '1px solid red', |