views:

107

answers:

3

So I want to store some information in localstorage of a browser when the page is refreshed or the user exits the page for future use. I figured that I'd use some Javascript to detect the event and store the state of the page visit.

Now my predicament is: shall i use Onbeforeunload or Onunload method to accomplish this objective?

Thanks

A: 

I believe that Onbeforeunload is a safe bet. It is possible for browser to not execute onunload script completely - for example, if you have alert in onunload, chrome would not show if you close the window.

VinayC
I tested Chrome v. 5 and 6 on Mac OS X and it does alert on OnUnload. I wish there was a matrix/chart that would list the DOM support for each browser. THanks.
pratyk
+3  A: 

Why not register it with both just to be on the safe side? Just have the listener do a check to see if the data's stored yet and exit early.

Here's a quick example using event properties:

window.onunload = window.onbeforeunload = (function(){

  var didMyThingYet=false;

  return function(){
    if (didMyThingYet) return;
    didMyThingYet=true;
    // do your thing here...
  }

}());

Or you could use attachEvent:

(function(){

  var didMyThingYet=false;

  function listener (){
    if (didMyThingYet) return;
    didMyThingYet=true;
    // do your thing here...
  }

  window.attachEvent("onbeforeunload", listener);
  window.attachEvent("onunload", listener);    

}());
no
That sounds like a doable idea to cover all bases. It'd be grand if you could point me to a snippet or a resource as far as the event listener and exiting early goes.
pratyk
thanks for the snippet. I tried it out across windows and mac os chrome/firefox and it seemed to cover all bases. A little more testing for Opera and IE9 beta should get me running.
pratyk
A: 

The proper place to do it is onunload. onbeforenload is meant to be used when you may need to stop the user from leaving the page, like when there is unsaved data.

I would be weary of writing code to cover all cases because you're not willing to test all cases. If you do your research and find that's it's too complicated across browsers, yeah, go ahead and do as much as you can. However, I believe there is no problem with doing local storage onunload. I actually did run into a problem, trying to send information to the server onunload. That did not happen reliably across browsers so I did end up with an approach that used both unload and beforeunload, but only after due diligence.

So my suggestion is do it onunload, check all your browsers.

Juan Mendes