xobs/fossasia-22-renode
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Check before calling blur on activeElement.

Browse Source 
It's possible for slides to be in a situation where the last clicked thing was an SVG before the tab/window loses focus. When returning, `.blur()` is called on the previously-active element, but can result in an exception.

This protects against that and will only call `.blur()` when `document.activeElement` supports it.
This commit is contained in:
Jordan Hofker 2015-02-19 16:09:08 -06:00
parent ce8ea84393
commit 76c5726c04
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
js/reveal.js
View File

@ -3872,7 +3872,10 @@
// If, after clicking a link or similar and we're coming back, // If, after clicking a link or similar and we're coming back,
// focus the document.body to ensure we can use keyboard shortcuts // focus the document.body to ensure we can use keyboard shortcuts
if( isHidden === false && document.activeElement !== document.body ) { if( isHidden === false && document.activeElement !== document.body ) {
// Not all elements support .blur() - SVGs among them.
if (typeof document.activeElement.blur === 'function') {
document.activeElement.blur(); document.activeElement.blur();
}
document.body.focus(); document.body.focus();
} }