[richfaces-issues] [JBoss JIRA] Reopened: (RF-10221) Error when opening a popup panel

[James G (JIRA)]

     [ https://issues.jboss.org/browse/RF-10221?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

James G reopened RF-10221:
--------------------------



I believe there's still a bug here.  If you set the  show parameter of rich:popupPanel (show="true") javascript is generated to call the suspect method: RichFaces.ui.PopupPanel.showPopupPanel.  It seems that the intension of this method was to queue the showing of the popup panel until the page has been loaded.  Instead, the show method is called immediately and an undefined is passed onto the queue (richfaces.Event.ready).  Please review this method.

> Error when opening a popup panel
> --------------------------------
>
>                 Key: RF-10221
>                 URL: https://issues.jboss.org/browse/RF-10221
>             Project: RichFaces
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: component-panels-layout-themes
>    Affects Versions: 4.0.0.Milestone5
>         Environment: myfaces 2.0.3, tomcat 6
>            Reporter: James G
>            Assignee: Gleb Galkin
>              Labels: popupPanel
>             Fix For: 4.0.0.Milestone6
>
>
> When I call show or hide for a popup panel I get the following error: "fn is undefined" from firefox.
> I believe the offening code is:  (in popupPanel.js)
>    showPopupPanel : function (id, opts, event) {
>       $(document).ready(richfaces.$(id).show());
>    },
>    hidePopupPanel : function (id, opts, event) {
>       $(document).ready(richfaces.$(id).hide());
>    }
> 		
> It should probably be something more like:
>    showPopupPanel : function (id, opts, event) {
>       $(document).ready( function() { richfaces.$(id).show() } );
>    },
>    hidePopupPanel : function (id, opts, event) {
>       $(document).ready( function(){ richfaces.$(id).hide() } );
>    }

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira