From 7396a20277eba466591dc8e8653e2a020b412d31 Mon Sep 17 00:00:00 2001 From: Dmitry Gozman Date: Mon, 18 Jul 2022 13:36:45 -0700 Subject: [PATCH] test: display:contents should be scrollable into view (#15765) --- .../elementhandle-scroll-into-view.spec.ts | 18 +++++++++++++++--- 1 file changed, 15 insertions(+), 3 deletions(-) diff --git a/tests/page/elementhandle-scroll-into-view.spec.ts b/tests/page/elementhandle-scroll-into-view.spec.ts index 20347e03c2..918afebbe0 100644 --- a/tests/page/elementhandle-scroll-into-view.spec.ts +++ b/tests/page/elementhandle-scroll-into-view.spec.ts @@ -60,9 +60,21 @@ it('should wait for display:none to become visible', async ({ page, server }) => await testWaiting(page, div => div.style.display = 'block'); }); -it('should wait for display:contents to become visible', async ({ page, server }) => { - await page.setContent('
Hello
'); - await testWaiting(page, div => div.style.display = 'block'); +it.fixme('should scroll display:contents into view', async ({ page, server }) => { + it.info().annotations.push({ type: 'issue', description: 'https://github.com/microsoft/playwright/issues/15034' }); + + await page.setContent(` +
+
+
+
Hello
+
+
+
+ `); + const div = await page.$('#target'); + await div.scrollIntoViewIfNeeded(); + expect(await page.$eval('#container', e => e.scrollTop)).toBe(350); }); it('should work for visibility:hidden element', async ({ page }) => {