[ http://issues.apache.org/jira/browse/BEEHIVE-863?page=all ]

Rich Feit updated BEEHIVE-863:
------------------------------

    Assign To: Carlin Rogers  (was: Rich Feit)

Hey Carlin, I've got a few comments/questions before submitting this patch...

ConfigurePopup.java:
---
- This block:
->      * @jsptagref.databindable false
->      * @jsptagref.attributesyntaxvalue <i>string_popupFunc</i>

should change to this:
->      * @jsptagref.databindable true
->      * @jsptagref.attributesyntaxvalue <i>boolean_passFormFields</i>

- What would you think of changing "passFormFields" to "updateFormFields"?

- getNearestForm(): could you pass this along to Daryl to get his review on 
that?  I feel like there might be subtleties that we need to watch out for (but 
I could be making this up :) ).

PopupSupport.java:
---
It looks like the regular popup function and the function that pops up WITH a 
form could be combined into one function, with the form passed as the 
(optional) last argument.  I think this would be cleaner -- what do you think?

> Passing a FormBean to a popup
> -----------------------------
>
>          Key: BEEHIVE-863
>          URL: http://issues.apache.org/jira/browse/BEEHIVE-863
>      Project: Beehive
>         Type: Improvement
>   Components: NetUI
>     Reporter: Jack Rudnick
>     Assignee: Carlin Rogers
>     Priority: Critical
>  Attachments: j863-first-patch.txt, j863-patch2.txt
>
> A popup is currently reached via a nested action, however, you can't pass a 
> FormBean (and more importantly, its values) from a parent action to the 
> nested action for use in the popup.  An example of how you may want to use 
> this would be:
>       >       1) parent form/page has a "To" text field in a form.
>       >       2) I type in "[EMAIL PROTECTED]"
>       >       3) click the button (netui:button popup=true) to trigger the 
> popup in the parent page
>       >       4) popup has its own form/page that has a "To" text field.
>       >       5) When the popup renders, the popups "To" field should be 
> prepopulated with the "hello [EMAIL PROTECTED]" value.
> You can currently pass the values of a popup to the parent, thus you should 
> be able to go the other way (pass the parent values to the popup) with a 
> given framework way other than constructing some javascript to do this in a 
> custom manner.  One reason why this probably doesn't currently work, is when 
> you click the netui:button it doesn't submit the parent form, thus the form 
> and values don't get persisted for use by the popup.
> This is currently a very limiting shortcoming of using popups.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to