Hi Maxim, Please do!
Thank you! On Nov 16, 2016 10:58 AM, "Maxim Solodovnik" <solomax...@gmail.com> wrote: > I believe all comments are now addressed, > Should I create PR to add AjaxDownload to wicket-extensions? both 7.x and > 8.x? > > On Fri, Nov 11, 2016 at 6:11 PM, Sven Meier <s...@meiers.net> wrote: > > > +1 on putting it into wicket-extensions. > > > > Sven > > > > > > Am 11.11.2016 um 10:54 schrieb Martin Grigorov: > > > >> One class + one .js might be too much for a new module but I don't want > >> all > >> the useless stuff in minis in my application too. > >> I still prefer this to be in Wicket core. > >> > >> Martin Grigorov > >> Wicket Training and Consulting > >> https://twitter.com/mtgrigorov > >> > >> On Fri, Nov 11, 2016 at 10:28 AM, Sven Meier <s...@meiers.net> wrote: > >> > >> I don't think this warrants a new module - just add it to > >>> wicketstuff-minis. > >>> > >>> Sven > >>> > >>> > >>> > >>> Am 11.11.2016 um 08:42 schrieb Martin Grigorov: > >>> > >>> Great work, Maxim! > >>>> > >>>> Let's move it to WicketStuff as a new module. > >>>> I'd like to add -examples module with different use cases so we can > see > >>>> whether the iframes leak or not for example. > >>>> Also I'd like to try to add events like beforeDownload/afterDownload > so > >>>> apps can show indicators that something is happening. > >>>> > >>>> Martin Grigorov > >>>> Wicket Training and Consulting > >>>> https://twitter.com/mtgrigorov > >>>> > >>>> On Wed, Nov 9, 2016 at 9:13 AM, Ernesto Reinaldo Barreiro < > >>>> reier...@gmail.com> wrote: > >>>> > >>>> Thanks for sharing. I just added a couple of comments/ > >>>> > >>>>> On Wed, Nov 9, 2016 at 4:23 AM, Maxim Solodovnik < > solomax...@gmail.com > >>>>> > > >>>>> wrote: > >>>>> > >>>>> Hello Martin, > >>>>> > >>>>>> sorry for the delay > >>>>>> > >>>>>> here is the repo: https://github.com/solomax/wicket-ajax-download > >>>>>> here is the commit with the ajax-download-via-iframe implementation: > >>>>>> https://github.com/solomax/wicket-ajax-download/commit/ > >>>>>> 407936d6f506aa047d9a12a3ecb7aa6c866eb052 > >>>>>> > >>>>>> Looking forward for your comments :) > >>>>>> > >>>>>> On Wed, Nov 9, 2016 at 5:02 AM, Martin Grigorov < > mgrigo...@apache.org > >>>>>> > > >>>>>> wrote: > >>>>>> > >>>>>> Hi Maxim, > >>>>>>> > >>>>>>> Do you have progress on this ? > >>>>>>> > >>>>>>> Martin Grigorov > >>>>>>> Wicket Training and Consulting > >>>>>>> https://twitter.com/mtgrigorov > >>>>>>> > >>>>>>> On Thu, Nov 3, 2016 at 9:46 AM, Maxim Solodovnik < > >>>>>>> solomax...@gmail.com > >>>>>>> wrote: > >>>>>>> > >>>>>>> I was hoping to get answer like: in 7.x you should use XXXX.xxx > :))) > >>>>>>> > >>>>>>>> Going to create example on github and will send it for review :) > >>>>>>>> > >>>>>>>> On Thu, Nov 3, 2016 at 3:43 PM, Martin Grigorov < > >>>>>>>> mgrigo...@apache.org > >>>>>>>> wrote: > >>>>>>>> > >>>>>>>> On Thu, Nov 3, 2016 at 9:40 AM, Maxim Solodovnik < > >>>>>>>> solomax...@gmail.com> > >>>>>>>> > >>>>>>> wrote: > >>>>>>> > >>>>>>>> It seems iframe is the only option :((( > >>>>>>>>> > >>>>>>>>>> Why so sad ? > >>>>>>>>>> > >>>>>>>>> iframe is a good option > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> here is the JS plugin wrapping this idea: > >>>>>>>>> > >>>>>>>>>> http://johnculviner.com/jquery-file-download-plugin- > >>>>>>>>>> for-ajax-like-feature-rich-file-downloads/ > >>>>>>>>>> going to perform additional search > >>>>>>>>>> > >>>>>>>>>> Thanks for the idea! > >>>>>>>>>> > >>>>>>>>>> On Thu, Nov 3, 2016 at 3:36 PM, Ernesto Reinaldo Barreiro < > >>>>>>>>>> reier...@gmail.com> wrote: > >>>>>>>>>> > >>>>>>>>>> or maybe use a hidden iframe to trigger download... > >>>>>>>>>> > >>>>>>>>>>> On Thu, Nov 3, 2016 at 9:28 AM, Ernesto Reinaldo Barreiro < > >>>>>>>>>>> reier...@gmail.com> wrote: > >>>>>>>>>>> > >>>>>>>>>>> I do not know if this is possible but > >>>>>>>>>>> > >>>>>>>>>>>> 1- Open a new tab > >>>>>>>>>>>> 2- Set location to download URL > >>>>>>>>>>>> 3- Close the new tab > >>>>>>>>>>>> > >>>>>>>>>>>> That way (maybe) page does not close WebSocket connection. It > >>>>>>>>>>>> > >>>>>>>>>>>> would > >>>>>>>>>>> > >>>>>>>>>> still > >>>>>>> > >>>>>>>> be "AJAX"... > >>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>>> On Thu, Nov 3, 2016 at 9:04 AM, Maxim Solodovnik < > >>>>>>>>>>>> > >>>>>>>>>>>> solomax...@gmail.com > >>>>>>>>>>> > >>>>>>>>>> wrote: > >>>>>>>>>> > >>>>>>>>>>> I'm afraid It would be not really Ajax ..... > >>>>>>>>>>>> > >>>>>>>>>>>>> On Thu, Nov 3, 2016 at 3:03 PM, Ernesto Reinaldo Barreiro < > >>>>>>>>>>>>> reier...@gmail.com> wrote: > >>>>>>>>>>>>> > >>>>>>>>>>>>> maybe open a second browser tab and do the download > >>>>>>>>>>>>> there... > >>>>>>>>>>>>> > >>>>>>>>>>>> On Thu, Nov 3, 2016 at 8:51 AM, Maxim Solodovnik < > >>>>>> > >>>>>>> solomax...@gmail.com> > >>>>>>>>>>>>> > >>>>>>>>>>>> wrote: > >>>>>>>>>>>> > >>>>>>>>>>>>> I'll try to create quick-start ASAP > >>>>>>>>>>>>>> > >>>>>>>>>>>>>>> On Thu, Nov 3, 2016 at 2:51 PM, Maxim Solodovnik < > >>>>>>>>>>>>>>> > >>>>>>>>>>>>>>> solomax...@gmail.com> > >>>>>>>>>>>>>> wrote: > >>>>>>>>>>>>>> > >>>>>>>>>>>>>>> AjaxDownload was for wicket 1.5.x (or maybe 6.x) > >>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>> maybe it can be enhanced to work without unload? > >>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>> On Thu, Nov 3, 2016 at 2:46 PM, Sven Meier < > >>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>> s...@meiers.net > >>>>>>>>>>>>>>> > >>>>>>>>>>>>>> wrote: > >>>>>>> > >>>>>>>> AjaxDownload changes the window location - the browser > >>>>>>>>>>>> > >>>>>>>>>>>>> probably > >>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>> prepares > >>>>>>>>>> > >>>>>>>>>>> unloading of the page, before opening the attached > >>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>> download > >>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>> in > >>>>>>> > >>>>>>>> a > >>>>>>>>>> > >>>>>>>>>> separate > >>>>>>>>>>> > >>>>>>>>>>>> window. > >>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>> Sven > >>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>> Am 03.11.2016 um 08:33 schrieb Martin Grigorov: > >>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>> Hi Maxim, > >>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>> I don't see any relation between those. > >>>>>>>>>>>>>>>>>> If it is easy to reproduce please create a > >>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>> quickstart. > >>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>> Martin Grigorov > >>>>>> > >>>>>>> Wicket Training and Consulting > >>>>>>>>>>>>>>>>>> https://twitter.com/mtgrigorov > >>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>> On Thu, Nov 3, 2016 at 4:16 AM, Maxim Solodovnik < > >>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>> solomax...@gmail.com > >>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>> wrote: > >>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>> Hello, > >>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>> Recently we found weird behavior of AjaxDownloader > >>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>> (similar > >>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>> to > >>>>>>>>> > >>>>>>>>>> this > >>>>>>>>>>> > >>>>>>>>>>>> [1] > >>>>>>>>>>>>>> > >>>>>>>>>>>>>>> one) > >>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>> For some reason at the moment download is initiated > >>>>>>>>>>>>>>>>>>> WebSocketBehavior::onClose is being called .... > >>>>>>>>>>>>>>>>>>> What is the reason for this? > >>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>> https://cwiki.apache.org/confluence/display/WICKET/ > >>>>>>>>>>>>>>>>>>> AJAX+update+and+file+download+in+one+blow > >>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>> -- > >>>>>>>>>>>>>>>>>>> WBR > >>>>>>>>>>>>>>>>>>> Maxim aka solomax > >>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>> ------------------------------ > >>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>> ------------------------------ > >>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>> --------- > >>>>>>>>> > >>>>>>>>>> To unsubscribe, e-mail: users-unsubscribe@wicket. > >>>>>>>>>>>>>> > >>>>>>>>>>>>>>> apache.org > >>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>> For additional commands, e-mail: > >>>>>>>>> > >>>>>>>>>> users-h...@wicket.apache.org > >>>>>>>>>>>>>>>> -- > >>>>>>>>>>>>>>>> WBR > >>>>>>>>>>>>>>>> Maxim aka solomax > >>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>> -- > >>>>>>>>>>>>>>> WBR > >>>>>>>>>>>>>>> Maxim aka solomax > >>>>>>>>>>>>>>> > >>>>>>>>>>>>>>> > >>>>>>>>>>>>>>> -- > >>>>>>>>>>>>>> Regards - Ernesto Reinaldo Barreiro > >>>>>>>>>>>>>> > >>>>>>>>>>>>>> > >>>>>>>>>>>>>> -- > >>>>>>>>>>>>> WBR > >>>>>>>>>>>>> Maxim aka solomax > >>>>>>>>>>>>> > >>>>>>>>>>>>> > >>>>>>>>>>>>> -- > >>>>>>>>>>>> Regards - Ernesto Reinaldo Barreiro > >>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>>> -- > >>>>>>>>>>> Regards - Ernesto Reinaldo Barreiro > >>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>>> -- > >>>>>>>>>> WBR > >>>>>>>>>> Maxim aka solomax > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> -- > >>>>>>>> WBR > >>>>>>>> Maxim aka solomax > >>>>>>>> > >>>>>>>> > >>>>>>>> -- > >>>>>> WBR > >>>>>> Maxim aka solomax > >>>>>> > >>>>>> ------------------------------------------------------------ > --------- > >>>>>> To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org > >>>>>> For additional commands, e-mail: users-h...@wicket.apache.org > >>>>>> > >>>>>> > >>>>>> > >>>>>> -- > >>>>> Regards - Ernesto Reinaldo Barreiro > >>>>> > >>>>> > >>>>> ------------------------------------------------------------ > --------- > >>> To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org > >>> For additional commands, e-mail: users-h...@wicket.apache.org > >>> > >>> > >>> > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org > > For additional commands, e-mail: users-h...@wicket.apache.org > > > > > > > -- > WBR > Maxim aka solomax >