From 7b3ce5eb56899328f3c65797b575ee74e9fec991 Mon Sep 17 00:00:00 2001 From: David Ortner Date: Fri, 27 Oct 2023 15:47:50 +0200 Subject: [PATCH] #1148@patch: Element.cloneNode() should not set scroll properties on the clone. --- packages/happy-dom/src/nodes/element/Element.ts | 4 ---- packages/happy-dom/test/nodes/element/Element.test.ts | 4 ---- 2 files changed, 8 deletions(-) diff --git a/packages/happy-dom/src/nodes/element/Element.ts b/packages/happy-dom/src/nodes/element/Element.ts index 35d47852a..3b807d157 100644 --- a/packages/happy-dom/src/nodes/element/Element.ts +++ b/packages/happy-dom/src/nodes/element/Element.ts @@ -379,10 +379,6 @@ export default class Element extends Node implements IElement { } (clone.tagName) = this.tagName; - clone.scrollLeft = this.scrollLeft; - clone.scrollTop = this.scrollTop; - clone.scrollWidth = this.scrollWidth; - clone.scrollHeight = this.scrollHeight; (clone.namespaceURI) = this.namespaceURI; return clone; diff --git a/packages/happy-dom/test/nodes/element/Element.test.ts b/packages/happy-dom/test/nodes/element/Element.test.ts index 4436ddff3..e216c20ff 100644 --- a/packages/happy-dom/test/nodes/element/Element.test.ts +++ b/packages/happy-dom/test/nodes/element/Element.test.ts @@ -1551,8 +1551,6 @@ describe('Element', () => { child.className = 'className'; (element.tagName) = 'tagName'; - (element.scrollLeft) = 10; - (element.scrollTop) = 10; // @ts-ignore element.namespaceURI = 'namespaceURI'; @@ -1562,8 +1560,6 @@ describe('Element', () => { const clone = element.cloneNode(false); const clone2 = element.cloneNode(true); expect(clone.tagName).toBe('tagName'); - expect(clone.scrollLeft).toBe(10); - expect(clone.scrollTop).toBe(10); expect(clone.namespaceURI).toBe('namespaceURI'); expect(clone.children.length).toEqual(0); expect(clone2.children.length).toBe(1);