mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor
synced 2024-11-25 14:56:20 +00:00
ea3fccccaa
We used to attempt to not break Firefox's bfcache, but this didn't really work very well, and it's not clear that avoiding bfcache breakage is even a good idea. It's more sensible and consistent to deliberately break bfcache while VE is active. On top of that, the Firefox people believe that our trick shouldn't even have worked to begin with: https://bugzilla.mozilla.org/show_bug.cgi?id=1102664 Because the onbeforeunload handler is removed when VE is deactivated, bfcache still works if you first click Read, then navigate away. Also clean up the management of the unload handler, using addEventListener() and removing the return value fallback code that is needed for beforeunload but not unload. For beforeunload this is harder to clean up because the addEventListener()-based API for returning a value isn't consistent across browsers. Change-Id: Ie4fe9ea3a59a54ba462733aa5e42bfc0ed5b15eb |
||
---|---|---|
.. | ||
ve.init.mw.MobileViewTarget.js | ||
ve.init.mw.ViewPageTarget.init.js | ||
ve.init.mw.ViewPageTarget.js |