The proposal to merge lp:~deryck/launchpad/mustache-model-undefined-891239 into 
lp:launchpad has been updated.

Description changed to:

We were getting an undefined error in Chromium with the new bug listings work 
because Chrome fires a popstate event on every page load, unlike every other 
browser.  popstate is what history tools use to know when you're moving forward 
and backward in history.  Other browsers only fire popstate when you press the 
back or forward buttons.

The work around was easy enough: inspect e.changed object to see if our history 
state data is in there.  When it's a page load this check doesn't pass, and the 
code does nothing.

The idea here is that we only do something on history:change when there's 
changed data to do something with.

For more details, see:
https://code.launchpad.net/~deryck/launchpad/mustache-model-undefined-891239/+merge/82463
-- 
https://code.launchpad.net/~deryck/launchpad/mustache-model-undefined-891239/+merge/82463
Your team Launchpad code reviewers is requested to review the proposed merge of 
lp:~deryck/launchpad/mustache-model-undefined-891239 into lp:launchpad.

_______________________________________________
Mailing list: https://launchpad.net/~launchpad-reviewers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~launchpad-reviewers
More help   : https://help.launchpad.net/ListHelp

Reply via email to