From 5216b77600b7af1f06c0b702472e20c159377d4a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Fri, 31 May 2013 17:47:48 +0200 Subject: [PATCH] dash: Make sure _clearDragPlaceholder() resets _dragPlaceholderPos The function currently only resets the placeholder position if there is a placeholder; this is not necessarily true, as the placeholder may be reset outside _clearDragPlaceholder(). If this happens, the placeholder will temporarily stop working for the "old" position (and permanently if it's the only position). Just reset the position unconditionally. https://bugzilla.gnome.org/show_bug.cgi?id=684618 --- js/ui/dash.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/js/ui/dash.js b/js/ui/dash.js index 0de00a29c..64aed3f63 100644 --- a/js/ui/dash.js +++ b/js/ui/dash.js @@ -799,8 +799,8 @@ const Dash = new Lang.Class({ if (this._dragPlaceholder) { this._dragPlaceholder.animateOutAndDestroy(); this._dragPlaceholder = null; - this._dragPlaceholderPos = -1; } + this._dragPlaceholderPos = -1; }, handleDragOver : function(source, actor, x, y, time) {