diff --git a/packages/combo-box/test/visual/lumo/combo-box.test.js b/packages/combo-box/test/visual/lumo/combo-box.test.js index fe67e960107..cd9470309ef 100644 --- a/packages/combo-box/test/visual/lumo/combo-box.test.js +++ b/packages/combo-box/test/visual/lumo/combo-box.test.js @@ -1,4 +1,4 @@ -import { fixtureSync, nextFrame } from '@vaadin/testing-helpers'; +import { fixtureSync, mousedown, nextFrame } from '@vaadin/testing-helpers'; import { sendKeys } from '@web/test-runner-commands'; import { visualDiff } from '@web/test-runner-visual-regression'; import '../common.js'; @@ -15,6 +15,13 @@ describe('combo-box', () => { element = fixtureSync('', div); }); + afterEach(() => { + // After tests which use sendKeys() the focus-utils.js -> isKeyboardActive is set to true. + // Click once here on body to reset it so other tests are not affected by it. + // An unwanted focus-ring would be shown in other tests otherwise. + mousedown(document.body); + }); + it('basic', async () => { await visualDiff(div, 'basic'); }); diff --git a/packages/combo-box/test/visual/lumo/screenshots/combo-box/baseline/ltr-loading.png b/packages/combo-box/test/visual/lumo/screenshots/combo-box/baseline/ltr-loading.png index 4abefcf0fb7..7bd9ea729e3 100644 Binary files a/packages/combo-box/test/visual/lumo/screenshots/combo-box/baseline/ltr-loading.png and b/packages/combo-box/test/visual/lumo/screenshots/combo-box/baseline/ltr-loading.png differ diff --git a/packages/combo-box/test/visual/lumo/screenshots/combo-box/baseline/ltr-opened.png b/packages/combo-box/test/visual/lumo/screenshots/combo-box/baseline/ltr-opened.png index b41360695b5..a5ab47e1519 100644 Binary files a/packages/combo-box/test/visual/lumo/screenshots/combo-box/baseline/ltr-opened.png and b/packages/combo-box/test/visual/lumo/screenshots/combo-box/baseline/ltr-opened.png differ diff --git a/packages/combo-box/test/visual/lumo/screenshots/combo-box/baseline/rtl-loading.png b/packages/combo-box/test/visual/lumo/screenshots/combo-box/baseline/rtl-loading.png index 7e0620ad554..ff5cb4cc6f1 100644 Binary files a/packages/combo-box/test/visual/lumo/screenshots/combo-box/baseline/rtl-loading.png and b/packages/combo-box/test/visual/lumo/screenshots/combo-box/baseline/rtl-loading.png differ diff --git a/packages/combo-box/test/visual/lumo/screenshots/combo-box/baseline/rtl-opened.png b/packages/combo-box/test/visual/lumo/screenshots/combo-box/baseline/rtl-opened.png index 13094838459..e8cd55e4f26 100644 Binary files a/packages/combo-box/test/visual/lumo/screenshots/combo-box/baseline/rtl-opened.png and b/packages/combo-box/test/visual/lumo/screenshots/combo-box/baseline/rtl-opened.png differ diff --git a/packages/date-picker/test/visual/lumo/date-picker.test.js b/packages/date-picker/test/visual/lumo/date-picker.test.js index 1aa754ac941..1784e7f1c2a 100644 --- a/packages/date-picker/test/visual/lumo/date-picker.test.js +++ b/packages/date-picker/test/visual/lumo/date-picker.test.js @@ -1,4 +1,4 @@ -import { fixtureSync } from '@vaadin/testing-helpers/dist/fixture.js'; +import { fixtureSync, mousedown } from '@vaadin/testing-helpers'; import { sendKeys } from '@web/test-runner-commands'; import { visualDiff } from '@web/test-runner-visual-regression'; import '../../../theme/lumo/vaadin-date-picker.js'; @@ -15,6 +15,13 @@ describe('date-picker', () => { element = fixtureSync('', div); }); + afterEach(() => { + // After tests which use sendKeys() the focus-utils.js -> isKeyboardActive is set to true. + // Click once here on body to reset it so other tests are not affected by it. + // An unwanted focus-ring would be shown in other tests otherwise. + mousedown(document.body); + }); + it('basic', async () => { await visualDiff(div, 'basic'); }); diff --git a/packages/date-picker/test/visual/lumo/screenshots/date-picker/baseline/ltr-dropdown.png b/packages/date-picker/test/visual/lumo/screenshots/date-picker/baseline/ltr-dropdown.png index 1a6e0f6fb9f..3b70d0a3337 100644 Binary files a/packages/date-picker/test/visual/lumo/screenshots/date-picker/baseline/ltr-dropdown.png and b/packages/date-picker/test/visual/lumo/screenshots/date-picker/baseline/ltr-dropdown.png differ diff --git a/packages/date-picker/test/visual/lumo/screenshots/date-picker/baseline/ltr-week-numbers.png b/packages/date-picker/test/visual/lumo/screenshots/date-picker/baseline/ltr-week-numbers.png index 1134f7bd398..33b6aecea17 100644 Binary files a/packages/date-picker/test/visual/lumo/screenshots/date-picker/baseline/ltr-week-numbers.png and b/packages/date-picker/test/visual/lumo/screenshots/date-picker/baseline/ltr-week-numbers.png differ diff --git a/packages/date-picker/test/visual/lumo/screenshots/date-picker/baseline/rtl-dropdown.png b/packages/date-picker/test/visual/lumo/screenshots/date-picker/baseline/rtl-dropdown.png index e589de4f94d..012f5074ad8 100644 Binary files a/packages/date-picker/test/visual/lumo/screenshots/date-picker/baseline/rtl-dropdown.png and b/packages/date-picker/test/visual/lumo/screenshots/date-picker/baseline/rtl-dropdown.png differ diff --git a/packages/date-picker/test/visual/lumo/screenshots/date-picker/baseline/rtl-week-numbers.png b/packages/date-picker/test/visual/lumo/screenshots/date-picker/baseline/rtl-week-numbers.png index c1d6b3901ec..028133c5e07 100644 Binary files a/packages/date-picker/test/visual/lumo/screenshots/date-picker/baseline/rtl-week-numbers.png and b/packages/date-picker/test/visual/lumo/screenshots/date-picker/baseline/rtl-week-numbers.png differ diff --git a/packages/time-picker/test/visual/lumo/time-picker.test.js b/packages/time-picker/test/visual/lumo/time-picker.test.js index 8d2e78fa97c..30d1492e253 100644 --- a/packages/time-picker/test/visual/lumo/time-picker.test.js +++ b/packages/time-picker/test/visual/lumo/time-picker.test.js @@ -15,6 +15,13 @@ describe('time-picker', () => { element.style.setProperty('--vaadin-time-picker-overlay-max-height', '300px'); }); + afterEach(() => { + // After tests which use sendKeys() the focus-utils.js -> isKeyboardActive is set to true. + // Click once here on body to reset it so other tests are not affected by it. + // An unwanted focus-ring would be shown in other tests otherwise. + mousedown(document.body); + }); + it('basic', async () => { await visualDiff(div, 'basic'); }); @@ -23,11 +30,6 @@ describe('time-picker', () => { await sendKeys({ press: 'Tab' }); await visualDiff(div, 'focus-ring'); - - // at this moment focus-utils.js -> isKeyboardActive is true. - // Click once here to reset it so other tests are not affected by it. - // A focus-ring would be shown in other tests otherwise. - mousedown(document.body); }); it('disabled', async () => {