From 88e1b58dfb6d23068a4c5157f8251d4ef3219d30 Mon Sep 17 00:00:00 2001 From: Blair McBride Date: Mon, 15 Apr 2013 10:52:24 +1200 Subject: [PATCH 1/9] Bug 861092 - When in customization mode, widgets aren't re-wrapped when dragging to the same position in an area. r=jaws. --- browser/modules/CustomizeMode.jsm | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/browser/modules/CustomizeMode.jsm b/browser/modules/CustomizeMode.jsm index 115abbd7a51..d1ee9ccd54a 100644 --- a/browser/modules/CustomizeMode.jsm +++ b/browser/modules/CustomizeMode.jsm @@ -531,6 +531,12 @@ CustomizeMode.prototype = { return; } + // Do nothing if the target was dropped onto itself (ie, no change in area + // or position). + if (draggedWrapper == targetNode) { + return; + } + // Is the target area the customization palette? If so, we have two cases - // either the originArea was the palette, or a customizable area. if (targetArea.id == kPaletteId) { From 70325929e1d30980ad0aa71582ce6b97d0b9cf61 Mon Sep 17 00:00:00 2001 From: Blair McBride Date: Mon, 15 Apr 2013 13:35:16 +1200 Subject: [PATCH 2/9] Bug 861702 - PanelUI sub-view container can end up showing multiple views. r=jaws. --- browser/base/content/panelUI.inc | 2 +- browser/modules/CustomizableUI.jsm | 4 ++++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/browser/base/content/panelUI.inc b/browser/base/content/panelUI.inc index 26f09b560fe..11515c4b263 100644 --- a/browser/base/content/panelUI.inc +++ b/browser/base/content/panelUI.inc @@ -67,7 +67,7 @@ whether or not it is shown. That's not good for our case, since we want to allow each subview to be uniquely sized. --> - +