[marinemap] push by perrygeo...@gmail.com - Fix panel close bug on 2011-10-19 18:23 GMT

0 views
Skip to first unread message

mari...@googlecode.com

unread,
Oct 19, 2011, 2:24:39 PM10/19/11
to marinema...@googlegroups.com
Revision: b439bebb9949
Author: Matthew Perry <perryg...@gmail.com>
Date: Wed Oct 19 11:26:14 2011
Log: Fix panel close bug
http://code.google.com/p/marinemap/source/detail?r=b439bebb9949

Modified:
/media/bookmarks/js/bookmarks.js
/media/common/js/layout/menu_items.js

=======================================
--- /media/bookmarks/js/bookmarks.js Wed Oct 19 10:53:27 2011
+++ /media/bookmarks/js/bookmarks.js Wed Oct 19 11:26:14 2011
@@ -14,7 +14,6 @@
};
var panel = lingcod.editors[0].panel;

-
function setupForm(form, options){
options = options || {};
var el = panel.getEl();
@@ -115,8 +114,8 @@
lingcod.setupForm = function(form){
setupForm(form);
}
- lingcod.menu_items.closeAll();
- $('.panelMask').hide();
+ //lingcod.menu_items.closeAll();
+ //$('.panelMask').hide();
panel.showUrl(url, panelOpts);
});
}
=======================================
--- /media/common/js/layout/menu_items.js Wed Oct 19 10:53:27 2011
+++ /media/common/js/layout/menu_items.js Wed Oct 19 11:26:14 2011
@@ -60,7 +60,12 @@
});
}
// lingcod.unmaskSidebar();
- $("#bookmark-close").click();
+
+ // This is to prevent multiple menu items from appearing at once
+ // BUT it causes some strange behavior: clicking on the bookmark
edit form panel
+ // causes the panel to close. Maybe having the bookmark panel PLUS
the data layers
+ // panel open at the same time *would* be desireable... "it's a
feature not a bug" ?
+ // $("#bookmark-close").click();
}

that.closeAll = closeAll;

Reply all
Reply to author
Forward
0 new messages