myfaces-users mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Adam Winer" <>
Subject Re: Dialog issue during ADF->Trinidad migration
Date Tue, 11 Sep 2007 20:31:43 GMT
There's two separate flags here:

- useWindow
- usePopup

If useWindow is false, that means we navigate the whole page
to the dialog.  Simple enough.

If useWindow is true, then we look at usePopup:  if it's false,
we want to show the dialog in a new browser window.
We use FredJSP so that we have a frameset around the
dialog view, needed to make sure that context is not lost
when you navigate within the dialog.

If usePopup is true, we use a DHTML dialog.  We don't
need FredJSP, since we're putting the dialog in an iframe,
and can directly navigate to the page.

Does this make sense?

What you're describing - "uses the URL returned from FredJSP.
getRedirectURL" - is intentional (and was the way things
worked back in ADF, FWIW).  What should be happening next
is that a frameset gets generated where the frame's source
is the URL of the desired page - so your dialog does show up.
Is that not working?

-- Adam

On 9/11/07, Bertrand, Shawn R <> wrote:
>  (Trinidad 1.0.2 – build from July – the current one).
> I've migrated our application to use Trinidad, and see some PPR issues
> that are likely ours, but for the most part everything is working as
> expected.  Except….
> We use the dialog framework extensively, and every time we attempt to
> display one a popup appears but uses the URL returned from
> FredJSP.getRedirectURL.  This happens because the code in CoreRenderKit,
> when constructing a DialogRequest object, calls usePopupForDialog to
> determine if the popup is supported.  Why wouldn't the passed-in usePopup
> setting be used?  Fortunately for me (at least for now), I added the
> "org.apache.myfaces.trinidadinternal.renderkit.USE_DIALOG_POPUP" context
> parameter to my web.xml and popups are now appearing (though they appear
> in a dhtml-looking layer instead of the traditional popup dialog (probably a
> good thing).
> Note: the Trinidad demo doesn't seem to need this context parameter to
> display dialogs.
> Thanks in advance,
> Shawn Bertrand
> Tyco Electronics Corporation

View raw message