Re: WebSocket close is being called if AjaxDownloader is used

2016-11-16 Thread Maxim Solodovnik
Done: https://github.com/apache/wicket/pull/190 https://github.com/apache/wicket/pull/191 On Wed, Nov 16, 2016 at 5:14 PM, Martin Grigorov wrote: > Hi Maxim, > > Please do! > > Thank you! > > On Nov 16, 2016 10:58 AM, "Maxim Solodovnik" wrote: >

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-16 Thread Martin Grigorov
Hi Maxim, Please do! Thank you! On Nov 16, 2016 10:58 AM, "Maxim Solodovnik" 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

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-16 Thread Maxim Solodovnik
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 wrote: > +1 on putting it into wicket-extensions. > > Sven > > > Am 11.11.2016 um 10:54 schrieb Martin Grigorov:

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-11 Thread Sven Meier
+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

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-11 Thread Maxim Solodovnik
In this case it might be good idea to create "full" wicketstuff project, then move it to core On Fri, Nov 11, 2016 at 4:54 PM, Martin Grigorov wrote: > 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

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-11 Thread 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

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-11 Thread Maxim Solodovnik
Ok, will try to do it in the beginning of next week Thanks for the pointer WBR, Maxim (from mobile, sorry for the typos) On Nov 11, 2016 16:28, "Sven Meier" wrote: > I don't think this warrants a new module - just add it to > wicketstuff-minis. > > Sven > > > Am 11.11.2016 um

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-11 Thread Sven Meier
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

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-10 Thread 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

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-09 Thread Ernesto Reinaldo Barreiro
Thanks for sharing. I just added a couple of comments/ On Wed, Nov 9, 2016 at 4:23 AM, Maxim Solodovnik wrote: > Hello Martin, > > sorry for the delay > > here is the repo: https://github.com/solomax/wicket-ajax-download > here is the commit with the

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-08 Thread Maxim Solodovnik
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

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-08 Thread Martin Grigorov
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 wrote: > I was hoping to get answer like: in 7.x you should use .xxx :))) > Going to create example

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-03 Thread Ernesto Reinaldo Barreiro
+1 for wicket-stuff On Thu, Nov 3, 2016 at 9:50 AM, Martin Grigorov wrote: > On Thu, Nov 3, 2016 at 9:46 AM, Maxim Solodovnik > wrote: > > > I was hoping to get answer like: in 7.x you should use .xxx :))) > > Going to create example on github

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-03 Thread Martin Grigorov
On Thu, Nov 3, 2016 at 9:46 AM, Maxim Solodovnik wrote: > I was hoping to get answer like: in 7.x you should use .xxx :))) > Going to create example on github and will send it for review :) > Good! Thanks! I think we should add AjaxDownload class either to Wicket core

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-03 Thread Maxim Solodovnik
I was hoping to get answer like: in 7.x you should use .xxx :))) Going to create example on github and will send it for review :) On Thu, Nov 3, 2016 at 3:43 PM, Martin Grigorov wrote: > On Thu, Nov 3, 2016 at 9:40 AM, Maxim Solodovnik > wrote: >

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-03 Thread Martin Grigorov
On Thu, Nov 3, 2016 at 9:40 AM, Maxim Solodovnik 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- >

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-03 Thread Maxim Solodovnik
It seems iframe is the only 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 <

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-03 Thread Martin Grigorov
On Thu, Nov 3, 2016 at 9:36 AM, Ernesto Reinaldo Barreiro < reier...@gmail.com> wrote: > or maybe use a hidden iframe to trigger download... > this should be simpler with the tabs some blocker may prevent it > > On Thu, Nov 3, 2016 at 9:28 AM, Ernesto Reinaldo Barreiro < > reier...@gmail.com>

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-03 Thread Ernesto Reinaldo Barreiro
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

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-03 Thread Ernesto Reinaldo Barreiro
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 wrote: > I'm afraid It

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-03 Thread Maxim Solodovnik
>> Maxim said: websocket.close() triggers the ajax download. Nope, sorry for my English I said For some reason at the moment download is initiated WebSocketBehavior::onClose is being called so user clicks "Download", and then, as the result WebSocket#onClose() is being called I would

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-03 Thread Martin Grigorov
On Thu, Nov 3, 2016 at 8:46 AM, Sven Meier wrote: > AjaxDownload changes the window location - the browser probably prepares > unloading of the page, before opening the attached download in a separate > window. > This could be the reason but it is the other way around of what

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-03 Thread Maxim Solodovnik
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 > wrote: > > > I'll try

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-03 Thread Ernesto Reinaldo Barreiro
maybe open a second browser tab and do the download there... On Thu, Nov 3, 2016 at 8:51 AM, Maxim Solodovnik wrote: > I'll try to create quick-start ASAP > > On Thu, Nov 3, 2016 at 2:51 PM, Maxim Solodovnik > wrote: > > > AjaxDownload was for wicket

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-03 Thread Maxim Solodovnik
I'll try to create quick-start ASAP On Thu, Nov 3, 2016 at 2:51 PM, Maxim Solodovnik 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 wrote: > >>

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-03 Thread Maxim Solodovnik
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 wrote: > AjaxDownload changes the window location - the browser probably prepares > unloading of the page, before opening the attached

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-03 Thread Sven Meier
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

Re: WebSocket close is being called if AjaxDownloader is used

2016-11-03 Thread 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 wrote: > Hello, > > Recently we found