Skip to content
This repository has been archived by the owner on May 11, 2021. It is now read-only.

Commit

Permalink
Comment out call to missing method.
Browse files Browse the repository at this point in the history
I don't think this double click functionality was widely used anyway.

Test plan:
Add compass, drag it around, scale it. No JS errors.

Auditors: emily
  • Loading branch information
Joel Burget committed Jan 8, 2015
1 parent a6de505 commit 8dceae1
Showing 1 changed file with 7 additions and 6 deletions.
13 changes: 7 additions & 6 deletions utils/constructions.js
Expand Up @@ -216,9 +216,10 @@ $.extend(KhanUtil, {
}
};

t.center.mouseTarget.dblclick(function() {
construction.removeTool(t, true);
});
// XXX(joel/emily)
// t.center.mouseTarget.dblclick(function() {
// construction.removeTool(t, true);
// });

$(t.perim[0]).css("cursor", "move");
$(t.perim[0]).bind(
Expand Down Expand Up @@ -380,7 +381,7 @@ $.extend(KhanUtil, {
//t.first.onMoveEnd(t.first.coord[0], t.first.coord[1]);
//t.second.onMoveEnd(t.second.coord[0], t.second.coord[1]);
};

$(t.center.mouseTarget[0]).bind("dblclick", function() {
construction.removeTool(t, true);
});
Expand Down Expand Up @@ -1032,8 +1033,8 @@ $.extend(KhanUtil, {
// Find angles to line points
var angles = [];
_.map(lines, function(tool) {
var angle1 = Math.atan2(tool.first.coord[1], tool.first.coord[0]) * 180 / Math.PI;
var angle2 = Math.atan2(tool.second.coord[1], tool.second.coord[0]) * 180 / Math.PI;
var angle1 = Math.atan2(tool.first.coord[1], tool.first.coord[0]) * 180 / Math.PI;
var angle2 = Math.atan2(tool.second.coord[1], tool.second.coord[0]) * 180 / Math.PI;
angles.push((angle1 - offsetAngle + 540 + 180 / n) % 360);
angles.push((angle2 - offsetAngle + 540 + 180 / n) % 360);
});
Expand Down

0 comments on commit 8dceae1

Please sign in to comment.