launchpad-reviewers team mailing list archive

[Deryck Hodge <deryck.hodge@xxxxxxxxxxxxx>]

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.