Simplify title: No need to observe
This commit is contained in:
committed by
GitHub
parent
38ab685143
commit
130c024c3e
@@ -41,7 +41,7 @@
|
|||||||
website,
|
website,
|
||||||
screen,
|
screen,
|
||||||
language,
|
language,
|
||||||
title,
|
title: document.title,
|
||||||
hostname,
|
hostname,
|
||||||
url: currentUrl,
|
url: currentUrl,
|
||||||
referrer: currentRef,
|
referrer: currentRef,
|
||||||
@@ -91,22 +91,6 @@
|
|||||||
history.replaceState = hook(history, 'replaceState', handlePush);
|
history.replaceState = hook(history, 'replaceState', handlePush);
|
||||||
};
|
};
|
||||||
|
|
||||||
const handleTitleChanges = () => {
|
|
||||||
const observer = new MutationObserver(([entry]) => {
|
|
||||||
title = entry && entry.target ? entry.target.text : undefined;
|
|
||||||
});
|
|
||||||
|
|
||||||
const node = document.querySelector('head > title');
|
|
||||||
|
|
||||||
if (node) {
|
|
||||||
observer.observe(node, {
|
|
||||||
subtree: true,
|
|
||||||
characterData: true,
|
|
||||||
childList: true,
|
|
||||||
});
|
|
||||||
}
|
|
||||||
};
|
|
||||||
|
|
||||||
const handleClicks = () => {
|
const handleClicks = () => {
|
||||||
document.addEventListener(
|
document.addEventListener(
|
||||||
'click',
|
'click',
|
||||||
@@ -226,7 +210,6 @@
|
|||||||
if (!initialized) {
|
if (!initialized) {
|
||||||
track();
|
track();
|
||||||
handlePathChanges();
|
handlePathChanges();
|
||||||
handleTitleChanges();
|
|
||||||
handleClicks();
|
handleClicks();
|
||||||
initialized = true;
|
initialized = true;
|
||||||
}
|
}
|
||||||
@@ -260,7 +243,6 @@
|
|||||||
|
|
||||||
let currentUrl = href;
|
let currentUrl = href;
|
||||||
let currentRef = referrer.startsWith(origin) ? '' : referrer;
|
let currentRef = referrer.startsWith(origin) ? '' : referrer;
|
||||||
let title = document.title;
|
|
||||||
let cache;
|
let cache;
|
||||||
let initialized;
|
let initialized;
|
||||||
let disabled = false;
|
let disabled = false;
|
||||||
|
|||||||
Reference in New Issue
Block a user