[gnome-shell] dnd: Fix used of undefined variables
- From: Colin Walters <walters src gnome org>
- To: svn-commits-list gnome org
- Cc:
- Subject: [gnome-shell] dnd: Fix used of undefined variables
- Date: Sat, 8 Aug 2009 14:09:36 +0000 (UTC)
commit 3c87d7674119038fa868c49fa874309e2d16a210
Author: Colin Walters <walters verbum org>
Date: Thu Aug 6 19:34:12 2009 -0400
dnd: Fix used of undefined variables
The variables this._yOffset and this._xOffset are included in the
drop coordinates, but as far as I can tell never defined. Looking
back on the commit that introduced this code, they weren't removed
from anywhere else either.
The drop coordinates seem correct without them, so just delete them.
js/ui/dnd.js | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
---
diff --git a/js/ui/dnd.js b/js/ui/dnd.js
index c7f753e..2eb5f78 100644
--- a/js/ui/dnd.js
+++ b/js/ui/dnd.js
@@ -133,8 +133,8 @@ _Draggable.prototype = {
// We can check the return value of the function and break the loop if it's true if we don't want
// to continue checking the parents.
target._delegate.handleDragOver(this.actor._delegate, actor,
- (stageX + this._dragOffsetX + this._xOffset - targX) / target.scale_x,
- (stageY + this._dragOffsetY + this._yOffset - targY) / target.scale_y,
+ (stageX + this._dragOffsetX - targX) / target.scale_x,
+ (stageY + this._dragOffsetY - targY) / target.scale_y,
event.get_time());
}
target = target.get_parent();
@@ -163,8 +163,8 @@ _Draggable.prototype = {
if (target._delegate && target._delegate.acceptDrop) {
let [targX, targY] = target.get_transformed_position();
if (target._delegate.acceptDrop(this.actor._delegate, actor,
- (dropX + this._xOffset - targX) / target.scale_x,
- (dropY + this._yOffset - targY) / target.scale_y,
+ (dropX - targX) / target.scale_x,
+ (dropY - targY) / target.scale_y,
event.get_time())) {
// If it accepted the drop without taking the actor,
// destroy it.
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]