Sspitting out the javascript for SecondLevel model does make little sense. So while I agree that changing application settings silently is not the best practice, neither is doing nothing. Most of the user will not be aware what multi window support does and that they should turn it off with SecondLevel mode, so I think we should do something to prevent writing the javascript.
-Matej Johan Compagner wrote: > Eelco did that: > > "do not set application settings automatically. 1) I don't agree with > it, because this multi window support is about more than just supporting > the back button 2) Even if we wanted to prevent that setting, it should > check and throw an exception if set, rather than quietly setting it and > leaving clients possibly with a big wtf if they find out this setting > suddenly change" > > and i don't agree again with that. > Because what the multi window does now is not what eelco wants. > The multi window check is now pure for back button support and not that > window session support what eelco also wants to have. > I also don't get where an exception should be thrown.. > > Again the multi window check is currently only there because whe had the > AccessStackPageMap that one needs it. > But the SecondLevelPageMap doesn't need it because pagemaps are not > needed anymore. > > When i am in the SecondLevel Mode i don't want to create new pagemaps if > a new browser window is detected... Never because new pagemaps (the > current concept) > is just a bad thing for the second level store.. I don't want new > pagemaps because that only means more use of memory and a potential mem > leak. > > The window support eelco wants is a bit different in my eyes. That is > forces by the developer not by the user when opening a new browser tab. > > johan > > > > > ------------------------------------------------------------------------ > > Using Tomcat but need to do more? Need to support web services, security? > Get stuff done quickly with pre-integrated technology to make your job easier > Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > > > ------------------------------------------------------------------------ > > _______________________________________________ > Wicket-develop mailing list > Wicket-develop@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/wicket-develop Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 _______________________________________________ Wicket-develop mailing list Wicket-develop@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-develop
