Refactoring and cleaning of code.

common/static/js/capa/drag_and_drop/draggables.js

@@ -16,47 +16,6 @@ define(['logme', 'update_input'], function (logme, updateInput) {
                 c1 += 1
             }
         }(0));
-
-        state.updateArrowOpacity();
-
-        $(document).mousemove(function (event) {
-            documentMouseMove(state, event);
-        });
-    }
-
-    function documentMouseMove(state, event) {
-        if (state.currentMovingDraggable !== null) {
-            state.currentMovingDraggable.iconEl.css(
-                'left',
-                event.pageX -
-                    state.baseImageEl.offset().left -
-                    state.currentMovingDraggable.iconWidth * 0.5
-                    - state.currentMovingDraggable.iconElLeftOffset
-            );
-            state.currentMovingDraggable.iconEl.css(
-                'top',
-                event.pageY -
-                    state.baseImageEl.offset().top -
-                    state.currentMovingDraggable.iconHeight * 0.5
-            );
-
-            if (state.currentMovingDraggable.labelEl !== null) {
-                state.currentMovingDraggable.labelEl.css(
-                    'left',
-                    event.pageX -
-                        state.baseImageEl.offset().left -
-                        state.currentMovingDraggable.labelWidth * 0.5
-                        - 9 // Account for padding, border.
-                );
-                state.currentMovingDraggable.labelEl.css(
-                    'top',
-                    event.pageY -
-                        state.baseImageEl.offset().top +
-                        state.currentMovingDraggable.iconHeight * 0.5 +
-                        5
-                );
-            }
-        }
     }
 
     function makeDraggableCopy(callbackFunc) {

common/static/js/capa/drag_and_drop/main.js

@@ -63,6 +63,8 @@ define(
             Scroller(state);
             Draggables.init(state);
 
+            state.updateArrowOpacity();
+
             // Update the input element, checking first that it is not filled with
             // an answer from the server.
             if (updateInput.check(state) === false) {

common/static/js/capa/drag_and_drop/state.js

@@ -8,7 +8,9 @@ define([], function () {
     return State;
 
     function State(problemId) {
-        return {
+        var state;
+
+        state = {
             'config': null,
 
             'baseImageEl': null,
@@ -33,6 +35,12 @@ define([], function () {
 
             'getUniqueId': getUniqueId
         };
+
+        $(document).mousemove(function (event) {
+            documentMouseMove(state, event);
+        });
+
+        return state;
     }
 
     function getUniqueId() {
@@ -53,6 +61,41 @@ define([], function () {
 
         return text;
     }
+
+    function documentMouseMove(state, event) {
+        if (state.currentMovingDraggable !== null) {
+            state.currentMovingDraggable.iconEl.css(
+                'left',
+                event.pageX -
+                    state.baseImageEl.offset().left -
+                    state.currentMovingDraggable.iconWidth * 0.5
+                    - state.currentMovingDraggable.iconElLeftOffset
+            );
+            state.currentMovingDraggable.iconEl.css(
+                'top',
+                event.pageY -
+                    state.baseImageEl.offset().top -
+                    state.currentMovingDraggable.iconHeight * 0.5
+            );
+
+            if (state.currentMovingDraggable.labelEl !== null) {
+                state.currentMovingDraggable.labelEl.css(
+                    'left',
+                    event.pageX -
+                        state.baseImageEl.offset().left -
+                        state.currentMovingDraggable.labelWidth * 0.5
+                        - 9 // Account for padding, border.
+                );
+                state.currentMovingDraggable.labelEl.css(
+                    'top',
+                    event.pageY -
+                        state.baseImageEl.offset().top +
+                        state.currentMovingDraggable.iconHeight * 0.5 +
+                        5
+                );
+            }
+        }
+    }
 });
 
 // End of wrapper for RequireJS. As you can see, we are passing