It's a bit a pitty that we have two seperate ways of registering and displaying flash messages. If I understand correctly, the difference with feedbackmessages is that feedbackmessages are cleared on every new request, regardless of whether they were displayed, and flashmessages are just sitting in the session until they are displayed - if ever. Furthermore, flash messages do not seem to be scoped in any way, so any flash panel that displays them removes the whole bunch from the session.
What problem were you trying to solve, and why did you choose to make this a seperate implementation from feedbackmessages - bloating the API in the process? Eelco On 3/18/06, Igor Vaynberg <[EMAIL PROTECTED]> wrote: > hey guys, i just added basic support for flash messages. basically flash > messages are messages that are persisted until they are displayed. kinda > like flash in ror. there is an example usage in wicket-phonebook. > > what i did was just add a list<string> into session and a FlashMessagesPanel > that displays+clears them. but this feels a bit disconnected from our > feedback message system. > > what id like to hear are some opninions on how and if the two should be > integrated. > > we could for example get rid of Component.flash() i added and instead add a > boolean param to info/error/blah methods to say whether or not that message > should be stored in flash scope (keeping the originals that default to false > of course). that would also let us give flash messages their error levels. > does that sound like a good idea? > > also should we integrate feedbackpanel to merge regular feedback messages > with flash messages? or keep the two panels separate. > > any feedback is welcome, > -Igor > > > > ------------------------------------------------------- This SF.Net email is sponsored by xPML, a groundbreaking scripting language that extends applications into web and mobile media. Attend the live webcast and join the prime developer group breaking into this new coding territory! http://sel.as-us.falkag.net/sel?cmd=lnk&kid0944&bid$1720&dat1642 _______________________________________________ Wicket-develop mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/wicket-develop
