Revert "Stop the pinball engine from hijacking the browser tab's favicon"
This reverts commit 4dd6d60457.
This commit is contained in:
@@ -61,32 +61,6 @@ export function usePinballInstance(
|
|||||||
const canvas = canvasRef.current;
|
const canvas = canvasRef.current;
|
||||||
if (!canvas || !webgl2Supported) return;
|
if (!canvas || !webgl2Supported) return;
|
||||||
|
|
||||||
// The engine sets a table-supplied image as the page favicon via a
|
|
||||||
// blob: URL (see vpinball.js's rel~='icon' link handling) — undesirable
|
|
||||||
// here, where the app's own favicon should stay put. Snapshot the
|
|
||||||
// current icon links up front and revert any blob: href the moment it
|
|
||||||
// appears, rather than patching the (compiled) engine itself.
|
|
||||||
const originalIconHrefs = new Map<HTMLLinkElement, string>();
|
|
||||||
document.querySelectorAll<HTMLLinkElement>("link[rel~='icon']").forEach((link) => {
|
|
||||||
originalIconHrefs.set(link, link.href);
|
|
||||||
});
|
|
||||||
const faviconObserver = new MutationObserver(() => {
|
|
||||||
document.querySelectorAll<HTMLLinkElement>("link[rel~='icon']").forEach((link) => {
|
|
||||||
if (!link.href.startsWith("blob:")) return;
|
|
||||||
const blobUrl = link.href;
|
|
||||||
const original = originalIconHrefs.get(link);
|
|
||||||
if (original) link.href = original;
|
|
||||||
else link.remove();
|
|
||||||
URL.revokeObjectURL(blobUrl);
|
|
||||||
});
|
|
||||||
});
|
|
||||||
faviconObserver.observe(document.head, {
|
|
||||||
childList: true,
|
|
||||||
subtree: true,
|
|
||||||
attributes: true,
|
|
||||||
attributeFilter: ["href"],
|
|
||||||
});
|
|
||||||
|
|
||||||
setStatus("loading");
|
setStatus("loading");
|
||||||
const startedAt = performance.now();
|
const startedAt = performance.now();
|
||||||
|
|
||||||
@@ -129,7 +103,6 @@ export function usePinballInstance(
|
|||||||
})();
|
})();
|
||||||
|
|
||||||
return () => {
|
return () => {
|
||||||
faviconObserver.disconnect();
|
|
||||||
disposedRef.current = true;
|
disposedRef.current = true;
|
||||||
const instance = instanceRef.current;
|
const instance = instanceRef.current;
|
||||||
instanceRef.current = null;
|
instanceRef.current = null;
|
||||||
|
|||||||
Reference in New Issue
Block a user