Author: amarkhel Date: 2010-12-28 10:07:08 -0500 (Tue, 28 Dec 2010) New Revision: 20830 Modified: trunk/ui/output/ui/src/main/resources/META-INF/resources/org.richfaces/popupPanel.js Log: https://issues.jboss.org/browse/RF-9741 Popup panel: attribute onhide doesn't work Modified: trunk/ui/output/ui/src/main/resources/META-INF/resources/org.richfaces/popupPanel.js =================================================================== --- trunk/ui/output/ui/src/main/resources/META-INF/resources/org.richfaces/popupPanel.js 2010-12-28 15:03:21 UTC (rev 20829) +++ trunk/ui/output/ui/src/main/resources/META-INF/resources/org.richfaces/popupPanel.js 2010-12-28 15:07:08 UTC (rev 20830) @@ -51,7 +51,7 @@ richfaces.ui.PopupPanel = function(id, options) { - $super.constructor.call(this,id); + $super.constructor.call(this,id); this.markerId = id; this.attachToDom(this.markerId); this.options = $.extend(this.options, defaultOptions, options || {}); @@ -499,9 +499,9 @@ var event = {}; event.parameters = opts || {}; - if (this.options && this.options.onhide) { - this.options.onhide(event); - } +// if (this.options && this.options.onhide) { +// this.options.onhide(event); +// } var forms = $("form", element); if (this.options.keepVisualState && forms) { @@ -754,12 +754,11 @@ $.extend(richfaces.ui.PopupPanel, { showPopupPanel : function (id, opts, event) { - - $(document).ready(richfaces.$(id).show()); + richface.Event.ready(richfaces.$(id).show()); }, hidePopupPanel : function (id, opts, event) { - $(document).ready(richfaces.$(id).hide()); + richface.Event.ready(richfaces.$(id).hide()); } });