? Success

User tests: Successful: Unsuccessful:

avatar hieblmedia
hieblmedia
27 Mar 2014

Keepalive without any library dependency.
I have tried to shrink the code as short as possible.
Should work from IE6 and all other modern Browsers.

avatar hieblmedia hieblmedia - open - 27 Mar 2014
avatar dongilbert
dongilbert - comment - 27 Mar 2014

:+1: Awesome!

avatar dbhurley
dbhurley - comment - 27 Mar 2014

This is great - we'll need to update the associated tests to it passes Travis. Can you update your PR with this?

avatar hieblmedia
hieblmedia - comment - 28 Mar 2014

Jup, i update the PR soon, when i am back on my dev workstation.

avatar hieblmedia
hieblmedia - comment - 29 Mar 2014

Test updated.

avatar dgt41
dgt41 - comment - 29 Mar 2014

Yes, vanilla JS!!! It works here

avatar Bakual
Bakual - comment - 29 Mar 2014

Can you open an issue tracker in JoomlaCode (http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemBrowse&tracker_id=8103) please? Unfortunately we still track testing there.

avatar dgt41
dgt41 - comment - 29 Mar 2014

I hope I didn't mess up. This should be the JoomlaCode link:
http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=33537

avatar Bakual
Bakual - comment - 29 Mar 2014

@dgt41 It's perfect, thanks!

avatar KISS-Web-Design
KISS-Web-Design - comment - 29 Mar 2014

I've had an open PR to replace MooTools with jQuery (5 months old, tested, passes Travis, etc).
#2494
vanilla JS is a better solution though, so please someone close mine too when this is merged.
Avoid leaving an orphaned out of date PR live :-)

avatar mbabker mbabker - change - 8 Apr 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-04-08 23:31:09
avatar mbabker mbabker - close - 8 Apr 2014
avatar mbabker mbabker - reference | - 8 Apr 14
avatar mbabker mbabker - merge - 8 Apr 2014
avatar mbabker mbabker - close - 8 Apr 2014
avatar Bakual Bakual - reference | 1b9b422 - 12 May 14

Add a Comment

Login with GitHub to post a comment